-
Notifications
You must be signed in to change notification settings - Fork 146
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
Update GSI_BINARY_SOURCE_DIR paths in modulefiles #579
Comments
@KateFriedman-NOAA , how do you suggest implementing the idea accepting gsi_ver from global-workflow fix.ver file to define the version in the path when run within GFS/global-workflow. See https://github.com/NOAA-EMC/global-workflow/blob/develop/versions/fix.ver#L14. in NOAA-EMC/GSI |
My initial thought would be similar to how the module versions get set. Something like this:
^ this would need to be tested, I parsed it together in this comment based on how we're doing similar things in global-workflow with incoming changes to add the initial capability to use version variables like we do in ops, example: https://github.com/KateFriedman-NOAA/global-workflow/blob/feature/version-files/modulefiles/module_base.hera.lua#L35. The |
Thank you @KateFriedman-NOAA for outlining what you have in mind. We may need to defer this until you or someone else has time to work on this. We do not have a GSI code manager to make and, more importantly, test the change. |
The GSI fix set was recently updated on disk and within global-workflow (see NOAA-EMC/global-workflow#1550). It was later noticed that there is a path in the GSI modulefiles (
GSI_BINARY_SOURCE_DIR
variable) that needs a matching update (see NOAA-EMC/global-workflow#1550 (comment)).Example in Hera modulefile: https://github.com/NOAA-EMC/GSI/blob/develop/modulefiles/gsi_hera.intel.lua#L29
The
GSI_BINARY_SOURCE_DIR
paths in the Hera, Orion, Jet, and WCOSS2 modulefiles need to be updated to use the newer20230112
fix set.Also, consider accepting
gsi_ver
from global-workflowfix.ver
file to define the version in the path when run within GFS/global-workflow. See https://github.com/NOAA-EMC/global-workflow/blob/develop/versions/fix.ver#L14.develop
after update is made (open global-workflow issue to make update).The text was updated successfully, but these errors were encountered: