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.

Sign up for GitHub

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

Jump to bottom

Add CentOS7 modules on Anvil #3260

Merged
merged 11 commits into from
Nov 5, 2019
Merged

Add CentOS7 modules on Anvil #3260

merged 11 commits into from
Nov 5, 2019

Conversation

amametjanov
Copy link
Member

Add CentOS7 modules on Anvil

[BFB]

@amametjanov amametjanov added Machine Files BFB PR leaves answers BFB Anvil labels Oct 28, 2019
@amametjanov
Copy link
Member Author

amametjanov commented Oct 28, 2019 •
edited
Loading

To test and run with new modules, please run with --compiler intel18 directly from the login nodes (ssh b512 not needed). Modules for Intel v.17.0.0 will be added as soon as they're installed. E.g.:

@blueslogin2:E3SM $ ./cime/scripts/create_test SMS.ne30_oECv3_ICG.A_WCYCL1850S_CMIP6

I didn't see errors in e3sm_integration runs.

@rljacob
Copy link
Member

rljacob commented Oct 29, 2019 •
edited
Loading

How does this relate to #3182 ?

@amametjanov
Copy link
Member Author

This has everything that #3182 has except for the constraint on "bad" thread counts. To check if this last piece is still an issue, I ran SMS_P18x4.ne4_ne4.FC5AV1C-L.anvil-centos7_intel18 11 times (1 initial 5-day run, 10 restart runs) and didn't get any NaN errors (like the ones previously on CentOS6+intel-17.0.0+mkl-2017.0.098). Besides CentOS7, this PR also includes newer intel/18.0.4-443hhug and intel-mkl/2018.4.274-jwaeshj modules.

@rljacob
Copy link
Member

rljacob commented Oct 30, 2019

Sounds like we can close #3182?

@amametjanov
Copy link
Member Author

Yes, after switching to CentOS7.

@amametjanov amametjanov force-pushed the sarich/anvil-centos7-modules branch from b4fe03f to 29256a8 Compare November 4, 2019 19:14
@amametjanov amametjanov assigned amametjanov and unassigned sarich Nov 4, 2019
amametjanov added a commit that referenced this pull request Nov 4, 2019
@amametjanov
Merge branch 'sarich/anvil-centos7-modules' into next (PR #3260)
39cb4a9 
Add CentOS7 modules on Anvil

[BFB]
amametjanov added a commit that referenced this pull request Nov 5, 2019
@amametjanov
Merge branch 'sarich/anvil-centos7-modules' (PR #3260)
7e0322d 
Add CentOS7 modules on Anvil

[BFB]
@amametjanov amametjanov merged commit 29256a8 into master Nov 5, 2019
@amametjanov amametjanov deleted the sarich/anvil-centos7-modules branch November 5, 2019 14:35
@amametjanov amametjanov mentioned this pull request Nov 7, 2019
Closed
@ndkeen
Copy link
Contributor

ndkeen commented Nov 7, 2019

Nice to have the fix for the failing DEBUG build, but why was it in a PR to add Anvil modules?

@ndkeen ndkeen mentioned this pull request Nov 7, 2019
Closed
@amametjanov
Copy link
Member Author

The errors were showing up when testing new modules.

rljacob pushed a commit that referenced this pull request Apr 21, 2021
@fischer-ncar
Merge pull request #3260 from ESMCI/fischer/maint-5.6_merge
af81112 
    Updates to workflow.
    Add namelist for minimum wind speed for atmOcn fluxes.

Test suite: scripts_regression_tests
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes [CIME Github issue #]

User interface changes?:

Update gh-pages html (Y/N)?:

Code review:
rljacob pushed a commit that referenced this pull request May 6, 2021
@fischer-ncar
Merge pull request #3260 from ESMCI/fischer/maint-5.6_merge
6114c7f 
    Updates to workflow.
    Add namelist for minimum wind speed for atmOcn fluxes.

Test suite: scripts_regression_tests
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes [CIME Github issue #]

User interface changes?:

Update gh-pages html (Y/N)?:

Code review:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@amametjanov amametjanov

Labels
Anvil BFB PR leaves answers BFB Machine Files
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@amametjanov @rljacob @ndkeen @sarich