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

Clean out module loads from within exrrfs_forecast.sh #623

Open
MatthewPyle-NOAA opened this issue Feb 7, 2025 · 3 comments
Open

Clean out module loads from within exrrfs_forecast.sh #623

MatthewPyle-NOAA opened this issue Feb 7, 2025 · 3 comments
Labels
EE2-specific Focused on improving EE2 compliance

Comments

@MatthewPyle-NOAA
Copy link
Contributor

Buried near the end with some netCDF format conversion is this line:

if ( ! time ( module purge ; module load intel szip hdf5 netcdf nco ; module list ; set -x ; ncks -3 --64 phy_data.nc4 phy_data.nc3) ) ; then

Is this special due to needing a special version of nco? @BenjaminBlake-NOAA @hu5970 Do you know the history of why this line is here?

@MatthewPyle-NOAA MatthewPyle-NOAA added the EE2-specific Focused on improving EE2 compliance label Feb 7, 2025
@BenjaminBlake-NOAA
Copy link
Contributor

@MatthewPyle-NOAA I'm not really sure. It looks like it is related to converting the phy_data netcdf file from netcdf4 to netcdf3, but I wonder if this logic is still needed in exrrfs_forecast.sh or it can be removed entirely. @ShunLiu-NOAA do you know?

@ShunLiu-NOAA
Copy link
Contributor

We had issues that the different NetCDF version were used for the different components (GSI, FV3 and Chgres). There was one of NetCDF version works best with FV3 for good I/O performance. Here is to ensure a consistent NetCDF format. But I don't know if we still need it after FMS lib updating.

@ShunLiu-NOAA
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EE2-specific Focused on improving EE2 compliance
Projects
Status: No status
Development

No branches or pull requests

3 participants