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

Setting namelist defaults with multiple physics suites? #77

Open
nusbaume opened this issue Sep 17, 2020 · 1 comment
Open

Setting namelist defaults with multiple physics suites? #77

nusbaume opened this issue Sep 17, 2020 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@nusbaume
Copy link
Collaborator

There are multiple CAMDEN namelist options whose default values depend on the physics option (phys) chosen. Normally this could be taken directly from the physics-suites config option, except currently the code allows for multiple physics suites to be listed even when there is only one model instance.

What should be done for the namelist defaults in the case of multiple physics suites but only a single host model instance? I can see a few possibilities:

  1. Simply disallow situations where the number of physics suites listed is greater than the number of instances (NINST), and assume each instance only has one associated physics suite.

  2. Have buildnml automatically choose a particular physics suite as the namelist default (e.g. the first suite listed in --physics-suites).

  3. Have the namelist defaults depend on the specific compset chosen (e.g. FKESSLER) instead of the physics suites listed.

Any thoughts or opinions? To be honest this might tie into a bigger question of what to do with multiple physics suites that conflict with each other in several different ways, such as containing schemes that represent the exact same physical processes (which could result in either double-counting or one scheme never running, both of which seem bad).

@nusbaume nusbaume added question Further information is requested discussion Issue used to generate a discussion amongst users/developers. labels Sep 17, 2020
@nusbaume nusbaume self-assigned this Sep 17, 2020
@nusbaume
Copy link
Collaborator Author

After discussion, it was decided that if there is more than one physics suite listed, then the user MUST state which physics suite is being used in user_nl_cam. If not, then buildnml will throw an error.

This feature will be brought in as part of the PR dealing with issue #44 .

@nusbaume nusbaume added enhancement New feature or request and removed discussion Issue used to generate a discussion amongst users/developers. question Further information is requested labels Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: To Do
Development

No branches or pull requests

1 participant