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

Icepack port to cori #258

Merged
merged 1 commit into from
May 24, 2019
Merged

Conversation

apcraig
Copy link
Contributor

@apcraig apcraig commented May 22, 2019

Port Icepack to cori, haswell nodes

  • Developer(s): tcraig

  • Are the code changes bit for bit, different at roundoff level, or more substantial? bit-for-bit

  • Does this PR create or have dependencies on CICE or any other models? none

  • Is the documentation being updated with this PR? (Y/N) N
    If not, does the documentation need to be updated separately at a later time? (Y/N) N

Note: "Documentation" includes information on the wiki and .rst files in doc/source/,
which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/.

  • Other Relevant Details:

See initial test results,

https://github.com/CICE-Consortium/Test-Results/wiki/ce05903831.cori.intel.190521.174418

Three tests are failing. I have had a quick look. In those cases, the runs are completing fine, the restarts just do not seem to be bit-for-bit. I ran one of the failures with debug turned on and then it passed the exact restart test, so this might be an issue with some code and the compiler. Using the same compiler flags, the full CICE test suite passes. I would like to defer the three failures for the time being.

@apcraig apcraig requested a review from eclare108213 May 22, 2019 16:25
@apcraig apcraig mentioned this pull request May 22, 2019
@eclare108213 eclare108213 merged commit 1591eec into CICE-Consortium:master May 24, 2019
lettie-roach pushed a commit to lettie-roach/Icepack that referenced this pull request Oct 18, 2022
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

Successfully merging this pull request may close these issues.

2 participants