Add optimization flag to fix irreproducibility on KNL/Intel #37
Conversation
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
reproducibility issues on some machines also replaced deprecated -FR flag with -free for free-form source
|
@philipwjones Thanks! I am testing my supported download cases for MPAS V6 this week, I can easily test this PR at the same time. |
mgduda
approved these changes
May 3, 2018
mark-petersen
approved these changes
May 14, 2018
mark-petersen
added a commit
that referenced
this pull request
May 14, 2018
added -align array64byte flag to optimized Intel builds to fix reproducibility issues on some machines (KNL) also replaced deprecated -FR flag with -free to denote free-form source B4B Tested on Cori, Edison at NERSC Replaces previous PR in MPAS-Dev/MPAS MPAS-Dev/MPAS#1520 though see discussion therein. Fixes previously reported bug in MPAS-Dev/MPAS MPAS-Dev/MPAS#1519
caozd999
pushed a commit
to caozd999/MPAS-Model
that referenced
this pull request
Jan 14, 2021
New Mesh: SOwISC12to60E2r4 Long name: SOwISC12to60kmL60E3SMv2r4 This version of the Southern Ocean Regionally Refined Mesh (SORRM) includes: * 12 km resolution around Antarctica, about 1000 km farther north than in revision 3 of the mesh * 45 km resolution at southern mid-latitudes * 30 km resolution at the equator and the north Atlantic * 60 km resolution in the north Pacific * 35 km resolution in the Arctic It matches the EC30to60 mesh except in the Southern Ocean and north Atlantic. The merge includes some reorganization of the SOwISC test cases within COMPASS: * the "resolution" has been renamed from `SO10to60wISC` to `SOwISC12to60` * a new test case `files_for_e3sm` has been added to create the initial condition, graph files, analysis region masks and mapping files, etc. The script that produces files for E3SM has been modified to use the short name of the mesh in output file names, rather than the long name. Initial condition, spin-up and files for E3SM are on Anvil at: ``` lcrc/group/e3sm/ac.xylar/mpas/spinup_expand_hr_SOwISC12to60E2r4/ocean/global_ocean/SOwISC12to60/ ```
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
added -align array64byte flag to optimized Intel builds to fix reproducibility issues on some machines (KNL)
also replaced deprecated -FR flag with -free to denote free-form source
B4B
Tested on Cori, Edison at NERSC
Replaces previous PR in MPAS-Dev/MPAS
MPAS-Dev/MPAS#1520
though see discussion therein.
Fixes previously reported bug in MPAS-Dev/MPAS
MPAS-Dev/MPAS#1519