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

follow atmos approach for config file loading #985

Open
juliasloan25 opened this issue Oct 1, 2024 · 0 comments
Open

follow atmos approach for config file loading #985

juliasloan25 opened this issue Oct 1, 2024 · 0 comments
Labels
enhancement New feature or request 🍃 leaf Issue coupled to a PR 💰 Grab Bag

Comments

@juliasloan25
Copy link
Member

ClimaAtmos and ClimaCoupler currently have slightly different approaches to setting up the config. Atmos reuses an existing config file if it's defined, or creates a new one if not. Coupler checks if we're running interactively and uses a default in that case, otherwise requires the user to specify a config file and job id. It might be nice to update the coupler approach to more closely align with that of atmos (see atmos code here).

Other things to think about:

  • rename interactive_debug.yml -> default_slabplanet.yml ? and create a default_amip.yml? might be useful for those just starting
  • extract config and parsed_args setup to a different file/function to increase clarity of driver code
@juliasloan25 juliasloan25 added enhancement New feature or request 💰 Grab Bag 🍃 leaf Issue coupled to a PR labels Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request 🍃 leaf Issue coupled to a PR 💰 Grab Bag
Projects
None yet
Development

No branches or pull requests

1 participant