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

Split off snapshot fields from registry #360

Open
jimmielin opened this issue Feb 11, 2025 · 0 comments
Open

Split off snapshot fields from registry #360

jimmielin opened this issue Feb 11, 2025 · 0 comments
Labels
enhancement New feature or request

Comments

@jimmielin
Copy link
Member

What is the feature/what would you like to discuss?

As physics schemes are ported more fields are being introduced into registry.xml that are solely for the purpose of reading these fields in from the snapshot (pbuf or tphysac/tphysbc) fields.

To avoid clutter in registry.xml, these could be split off into a separate file, and the generated code for handling those might also potentially be separated into separate modules.

Other notes from CAM-SIMA planning meeting about this:

  • @nusbaume pointed out that a snapshot registry may still be needed in the future for single-column CAM applications, as these model runs do not provide the full model state.
  • @cacraigucar suggested adding a line in registry.xml now to split off the snapshot variables from the rest.

Is there anyone in particular you want to be part of this conversation?

@cacraigucar @peverwhee @nusbaume

Will this change (regression test) answers?

No

Will you be implementing this enhancement yourself?

Any Software Engineer can do this

@jimmielin jimmielin added the enhancement New feature or request label Feb 11, 2025
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