-
Notifications
You must be signed in to change notification settings - Fork 48
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 alternate FMS & CRTM versions from unified-dev #548
Comments
Regarding 1 - this can be removed. Regarding 2 - no idea. But with a bit of luck we'll be able to remove the release-jcsda version altogether in the next few weeks to months. |
@AlexanderRichert-NOAA I don't think we'll be able to avoid multiple CRTM versions for a very long time (until everyone uses CRTM v3 - GFS v18 for the UFS?). For FMS, I am hopeful that once NOAA-GFDL/FMS#1225 is merged, we have the opportunity to remove the different FMS versions and work with the latest FMS release in all our environments. |
The FMS part of this issue is blocked by NOAA-GFDL/FMS#1225 |
Kicking the can down the road to spack-stack-1.7.0 ... |
@BenjaminTJohnson We added you here to check what the timeline/prospect of moving to crtm v3 everywhere is. If that's not feasible within the next year or so I suggest with close this issue after the FMS unification is done (hopefully soon!) |
@climbfuji do you mean to ask when we're deprecating v2.x versions and only supporting v3.x? |
For spack-stack, we need to know when we can stop installing crtm@v2 with each release - probably not for a while because some operational implementations will use crtm@v2? |
@climbfuji yes, my best guess would be 2-3 years before we stop supporting v2.4.0 operationally. |
We can now remove fms@release-jcsda from the templates and the JEDI packages, and have our package match spack develop. @srherbener Can you take care of this, please? |
The fms divergence is now history ... all that remains is crtm. |
For spack-stack-1.9.0, we still need different versions of CRTM. I will move this to the following release of spack-stack, with some luck everbody will be on the same version of CRTM 3.x.y by then. |
Editing this issue to cover both FMS and CRTM.
FMS and CRTM currently have alternate versions in unified-dev, which, among other things, can trigger concretization bugs. It would be lovely to get to a point where we can see unify: true.
Relatedly:
The following code appears in fms/package.py:
The
with when()
construct does not function as a conditional statement, so this should be corrected as soon as possible.The text was updated successfully, but these errors were encountered: