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

Discrepancy in NetCDF Version used in Instructions for Jet Install and Runtime Modules #40

Closed
dmwright526 opened this issue Oct 21, 2020 · 2 comments

Comments

@dmwright526
Copy link
Collaborator

On the installation instructions for Jet (https://github.com/ufs-community/ufs-srweather-app/blob/master/docs/README_jet_intel.txt) it uses netcdf/4.6.1, but in the module files for at least make_grid and make_orog use netcdf/4.7.0. This discrepancy results in runtime failures for at least make_grid and make_orog in the workflow.

Switching the module files for the above two tasks to netcdf/4.6.1 does solve the issue, but it might make more sense to adjust the installation instructions to use 4.7.0. I am not sure if 4.6.1 is specifically needed during the installation process.

@mkavulich
Copy link
Collaborator

Thanks for this report David; the newest README instructions that build Jet have been updated but not yet pushed to the repository. These updated instructions will be pushed to this repo once this NCEPLIBS update is merged: NOAA-EMC/NCEPLIBS#148

If you need to build on Jet right now, updated instructions can be found on my fork (https://github.com/mkavulich/ufs-srweather-app/tree/build_READMEs/docs). But I'd recommend waiting until the official update if possible.

@dmwright526
Copy link
Collaborator Author

Thanks for the information, @mkavulich ! I caught this during a fresh build on Jet, but it sounds like this has already been corrected in the future PR. Thanks!

mkavulich added a commit that referenced this issue Nov 2, 2020
## DESCRIPTION OF CHANGES: 
The new cmake build requires the user to set up the environment specific to their platform. These requirements are now documented in a series of README_* files that the user can follow along with for setting their environment.

In addition, this PR would remove the old build system scripts to avoid confusion.

## TESTS CONDUCTED: 
Built successfully on all of the platforms with associated README files, save Cheyenne (gnu) and MacOSX. I will iterate on these instructions until I can get a successful build.

Out of an abundance of caution, also ran a series of end-to-end run tests on Hera; all of which ran successfully:

 * grid_RRFS_CONUS_13km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v15p2
 * grid_RRFS_CONUS_13km_ics_HRRRX_lbcs_RAPX_suite_RRFS_v1beta
 * grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v15p2
 * grid_RRFS_CONUS_25km_ics_HRRRX_lbcs_RAPX_suite_RRFS_v1beta
 * grid_RRFS_CONUS_3km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v15p2
 * grid_RRFS_CONUS_3km_ics_HRRRX_lbcs_RAPX_suite_RRFS_v1beta

## ISSUES: 
Resolves #40, #37
christinaholtNOAA pushed a commit to christinaholtNOAA/ufs-srweather-app that referenced this issue Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants