-
Notifications
You must be signed in to change notification settings - Fork 119
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 build environment files for Hera, Orion, WCOSS with latest hpc-stack #138
Merged
Merged
Changes from 12 commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
f94c85e
Update the build enviroment
chan-hoo d3e5a23
Update the build environment for wcoss dell
chan-hoo d6fe3c6
Update the build environment for wcoss cray
chan-hoo 4fc7c1c
Update the build environment for Hera
chan-hoo 6477e91
Update hash of regional workflow
chan-hoo 7a32f67
Update build environment for Hera
chan-hoo dfad488
Update build environment for Orion
chan-hoo 28fd511
Re-order modules
chan-hoo f416091
Update module list for wcoss dell
chan-hoo 93db64e
Update module list for wcoss dell
chan-hoo 38fdb07
back to the original hash
chan-hoo 264d533
Rearrange modules in alphabetical order
chan-hoo 232a32a
Remove unnecessary modules
chan-hoo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,40 +1,49 @@ | ||
#Setup instructions for MSU Orion using Intel-18.0.5 (bash shell) | ||
|
||
module load contrib noaatools | ||
module purge | ||
|
||
module load cmake/3.17.3 | ||
module load contrib noaatools | ||
module load python/3.7.5 | ||
|
||
module use /apps/contrib/NCEP/test/hpc-stack-nco/modulefiles/stack | ||
|
||
module load hpc/1.0.0-beta1 | ||
### hpc-stack ### | ||
module use /apps/contrib/NCEP/libs/hpc-stack/modulefiles/stack | ||
module load hpc/1.1.0 | ||
module load hpc-intel/2018.4 | ||
module load hpc-impi/2018.4 | ||
|
||
module load jasper/2.0.22 | ||
module load zlib/1.2.11 | ||
module load png/1.6.35 | ||
module load hdf5/1.10.6 | ||
module load netcdf/4.7.4 | ||
module load pio/2.5.1 | ||
module load esmf/8_1_0_beta_snapshot_27 | ||
module load bacio/2.4.1 | ||
module load crtm/2.3.0 | ||
module load gfsio/1.4.1 | ||
module load g2/3.4.1 | ||
module load g2tmpl/1.9.1 | ||
module load ip/3.3.3 | ||
module load jasper/2.0.22 | ||
module load nemsio/2.5.2 | ||
module load png/1.6.35 | ||
module load sfcio/1.4.1 | ||
module load sigio/2.3.2 | ||
module load sp/2.3.3 | ||
module load wgrib2/2.0.8 | ||
module load wrf_io/1.1.1 | ||
module load w3emc/2.7.3 | ||
module load w3nco/2.4.1 | ||
module load upp/10.0.0 | ||
module load zlib/1.2.11 | ||
|
||
module load gfsio/1.4.1 | ||
module load sfcio/1.4.1 | ||
### Additional modules ### | ||
module load esmf/8_1_0_beta_snapshot_27 | ||
module load hdf5/1.10.6 | ||
module load landsfcutil/2.4.1 | ||
module load nccmp/1.8.7.0 | ||
module load nemsiogfs/2.5.3 | ||
module load wgrib2/2.0.8 | ||
module load netcdf/4.7.4 | ||
module load upp/10.0.0 | ||
module load wrf_io/1.1.1 | ||
chan-hoo marked this conversation as resolved.
Show resolved
Hide resolved
|
||
module load w3emc/2.7.3 | ||
|
||
|
||
export CMAKE_C_COMPILER=mpiicc | ||
export CMAKE_CXX_COMPILER=mpiicpc | ||
export CMAKE_Fortran_COMPILER=mpiifort | ||
export CMAKE_Platform=orion.intel | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Should # UFS_UTILS be listed above these two modules? HPSS is only used for the exregional_get_extrn_mdl_files.sh script and cmake is for everything (comment applies to all environment files).
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.
@JeffBeck-NOAA, The comment means the order of UFS_UTILS -> ufs-weather-model -> EMC_post (order in Externals.cfg). It doesn't mean a specific component. This is the reason why I put '+' before EMC_post or ufs-weather-model in comments (for example, # + EMC_post). I used it as an indicator of 'in addition'. If you think these comments make users confused, I'll remove all the comments and rearrange the modules in the alphabetical order.
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.
Thanks, @chan-hoo. I'm honestly still confused, but it could just be me. I was thinking they were block headers that represent which modules apply to which App component. How do they represent the Externals.cfg file?
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.
@JeffBeck-NOAA, I've removed all the comments about components and rearranged modules only in alphabetical order. I'll test building on each machine again.
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.
@JeffBeck-NOAA, completed!
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.
Thank you @chan-hoo! This looks good to me. Going to approve, but would like to have @mkavulich's review as well.