-
Notifications
You must be signed in to change notification settings - Fork 64
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
Remove run phase requirement from capgen #543
Comments
Out of curiosity, what is the use case for this? I think it would be nice to document something here. |
The UFS Weather Model has a few schemes that only have a |
Thanks. I wonder if this is an optimal solution or if we should instead modify the idea of |
Here's the solution I used to get around this issue with Capgen in the CCPP SCM, dustinswales@934f1f5. |
Here we include a small fix to allow for schemes without run phases. Capgen assumes that all schemes have a run phase, which is not true in practice. During the "scheme analyze" section, if a scheme doesn't contain a run phase we set a flag to true (no_run_phase). Then this flag is queried later in the "scheme write" section. There are no interface changes. Addresses #543 Testing has been expanded to contain a scheme w/o a run phase.
Resolved by #548 |
Description
Capgen should no longer require a run phase, to allow for more flexibility.
Solution
I don't have a full solution - but updates will be needed to ccpp_suite.py and possibly suite_objects.py to remove some of the default behavior/handling of groups/run phases.
The text was updated successfully, but these errors were encountered: