-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Add develop version to ufs-weather-model (major updates) #39265
Add develop version to ufs-weather-model (major updates) #39265
Conversation
@becker33 I took out the architecture settings since I don't think we need them for the spack recipe anyway (they're for configuring specific sites which we don't care about), so this way it'll just configure those settings through SPACK_TARGET_ARGS. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm happy to take over reviewing for @becker33. My B.S. was in Earth Science and I'm now working on ML weather forecasting, so I have some interest in seeing UFS improved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any other red flags. As long as it builds for you and you're committed to helping fix any bugs that other users report, it should be fine. Hoping we can see more regular releases in the future!
Change requests have been resolved.
This PR adds a develop version to the ufs-weather-model recipe, which includes major updates to variants and dependencies. Tested with ATM, S2SW, and ATMAERO configurations to cover all dependencies.
Fixes JCSDA/spack-stack#377