Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default user setting in CHILLED is passed over even if selected user is different #31

Open
measrainsey opened this issue Oct 22, 2024 · 0 comments · Fixed by #32 · May be fixed by #29
Open

Default user setting in CHILLED is passed over even if selected user is different #31

measrainsey opened this issue Oct 22, 2024 · 0 comments · Fixed by #32 · May be fixed by #29
Assignees
Labels
bug Something isn't working chilled PRs and issues related to the CHILLED model

Comments

@measrainsey
Copy link
Contributor

measrainsey commented Oct 22, 2024

In the config object for CHILLED, a user is specified as such:

user: Literal["ALE", "ED", "MEAS", "MEAS_EBRO", "MEAS_UNICC"] = "MEAS_UNICC"

Further down in the config, the user is used to define the paths used in the CHILLED module:

#: Paths settings by user
project_path: str = str(DICT_USER_SETTINGS[user]["project_path"])
dle_path: str = str(DICT_USER_SETTINGS[user]["dle_path"])
message_region_file: str = str(DICT_USER_SETTINGS[user]["message_region_map_file"])
isimip_bias_adj_path: str = str(DICT_USER_SETTINGS[user]["isimip_bias_adj_path"])
isimip_ewemib_path: str = str(DICT_USER_SETTINGS[user]["isimip_ewembi_path"])
chunk_size = DICT_USER_SETTINGS[user]["chunk_size"]

However, currently, when a user that is not the default is called, the paths being set are still the ones associated with the default user.

On my immediate TODO is to change the path setting to be in a separate function not within the config.

@measrainsey measrainsey added bug Something isn't working chilled PRs and issues related to the CHILLED model labels Oct 22, 2024
@measrainsey measrainsey self-assigned this Oct 22, 2024
@measrainsey measrainsey linked a pull request Oct 31, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working chilled PRs and issues related to the CHILLED model
Projects
None yet
1 participant