-
Notifications
You must be signed in to change notification settings - Fork 111
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
Use spack-stack-1.6.0 environment, in accordance with the current UFS-WM stack #1028
base: develop
Are you sure you want to change the base?
Use spack-stack-1.6.0 environment, in accordance with the current UFS-WM stack #1028
Conversation
@DavidHuber-NOAA - let us know if this is a problem for the global workflow. |
@natalie-perlin - I only have access to the Tier 1 machines (Hera, Jet, Orion, Hercules and WCOSS2). You will need to test on the other machines. |
On Hera, when I compile and run the
'develop' uses Gnu v9.2. This branch uses v13.3.0. |
Will take a look. Some additional env. variables and or/linking flags could be required (as for other repos), looking into this. |
@GeorgeGayno-NOAA - updated the build.hera.gnu.lua modulefile, adding cpp flags, library paths and preloaded library to address the runtime issues. The following has been added to the modulefile:
Building the executable was done as following:
The log file is attached: llog.build.hera.gnu13.3.txt |
That fixed it. I was able to run the unit tests and |
a9559d8
to
2d42c42
Compare
@natalie-perlin - be sure to merge in the latest updates from 'develop'. |
@DavidHuber-NOAA - are these changes OK with the global workflow? |
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.
@GeorgeGayno-NOAA Yes, these are fine for the global workflow.
Thanks @natalie-perlin for the updates! Approve.
…ity#1027) Previously, the tests ran in serial and some parts were run from the command line. This speeds up the processing and places all the test functions in the batch queue. The functionality was not changed. Fixes ufs-community#1016.
@DavidHuber-NOAA @GeorgeGayno-NOAA - |
DESCRIPTION OF CHANGES:
Modulefiles are updated to use spack-stack-1.6.0 environment
as currently used by UFS-WM and UFS-SRW App.
The main difference of the software package versions as related to the UFS_UTILS is an updated version of g2/3.5.1. It would further enable implement changes to the UFS_UTILS that require this higher version of g2, such as reading large grib2 files.
Files adapted for platforms and compilers, except for Stampede (no longer supported or used by UFS as Tier 1 platforms) and WCOSS2.
TESTS CONDUCTED:
Describe any additional tests performed.
DEPENDENCIES:
DOCUMENTATION:
ISSUE:
This PR is the first step to contribute in resolving the issue ufs-community/ufs-srweather-app#1191 that requires reading large grib2 files.