-
Notifications
You must be signed in to change notification settings - Fork 22
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
Quick bugfix to create super yamls before expr
directories
#337
Quick bugfix to create super yamls before expr
directories
#337
Conversation
562880a
Added one more change to this PR to move the utilities for updating ctest inputs/refs into |
@hu5970 @guoqing-noaa Please review this PR if you have a chance. |
PASSED on jet started build_and_test on jet at UTC time: Mon Mar 24 03:51:30 UTC 2025
workdir: /lfs5/BMC/wrfruc/rrfsbot/PRs_RDASApp/337 |
PASSED on hera started build_and_test on hera at UTC time: Mon Mar 24 03:53:48 UTC 2025
workdir: /scratch1/NCEPDEV/fv3-cam/rrfsbot/PRs_RDASApp/337 |
PASSED on hercules started build_and_test on hercules at UTC time: Mon Mar 24 03:50:54 UTC 2025
workdir: /work/noaa/wrfruc/rrfsbot/PRs_RDASApp/337 |
Bug Fix Description
This PR resolves error messages when building RDASApp that occur due to the
expr/
directories being set up before the super yamls are created. This was preventing the super yamls from being copied into theexpr/mpas_2024052700
andexpr/fv3_2024052700
directories.Description of Current Behavior:
Running
./build.sh
produces errors like:Description of Fixed Behavior:
Now
gen_yaml_ctest.sh
is run beforelink_mpasjedi_expr.sh
andlink_fv3jedi_expr.sh
to prevent this error and correctly copy the super yamls into the expr directories.Issue(s) addressed
None
Dependencies (if applicable)
None
Checklist