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

Remove alternate FMS & CRTM versions from unified-dev #548

Open
AlexanderRichert-NOAA opened this issue Apr 17, 2023 · 10 comments
Open

Remove alternate FMS & CRTM versions from unified-dev #548

AlexanderRichert-NOAA opened this issue Apr 17, 2023 · 10 comments
Assignees
Labels
bug Something is not working INFRA JEDI Infrastructure NAVY United States Naval Research Lab

Comments

@AlexanderRichert-NOAA
Copy link
Collaborator

AlexanderRichert-NOAA commented Apr 17, 2023

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:

    with when("@release-jcsda"):
        git = "https://github.com/JCSDA/fms.git"

The with when() construct does not function as a conditional statement, so this should be corrected as soon as possible.

@climbfuji
Copy link
Collaborator

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.

@climbfuji climbfuji added the INFRA JEDI Infrastructure label Apr 17, 2023
@AlexanderRichert-NOAA AlexanderRichert-NOAA changed the title FMS/jcsda version Remove alternate FMS & CRTM versions from unified-dev Jun 1, 2023
@climbfuji
Copy link
Collaborator

@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.

@climbfuji
Copy link
Collaborator

The FMS part of this issue is blocked by NOAA-GFDL/FMS#1225

@climbfuji
Copy link
Collaborator

climbfuji commented Dec 7, 2023

Kicking the can down the road to spack-stack-1.7.0 ...

@climbfuji climbfuji added the NAVY United States Naval Research Lab label May 13, 2024
@climbfuji
Copy link
Collaborator

@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!)

@BenjaminTJohnson
Copy link

@climbfuji do you mean to ask when we're deprecating v2.x versions and only supporting v3.x?

@climbfuji
Copy link
Collaborator

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?

@BenjaminTJohnson
Copy link

@climbfuji yes, my best guess would be 2-3 years before we stop supporting v2.4.0 operationally.

@climbfuji
Copy link
Collaborator

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?

@climbfuji
Copy link
Collaborator

The fms divergence is now history ... all that remains is crtm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working INFRA JEDI Infrastructure NAVY United States Naval Research Lab
Projects
Development

No branches or pull requests

4 participants