Use reproducibility flags #14
Conversation
util/make_dir/make.nci
Outdated
| NCI_DEBUG_FLAGS = -g3 -O0 -debug all -check all | ||
| F90FLAGS_1 = $(NCI_INTEL_FLAGS) $(NCI_REPRO_FLAGS) $(NCI_DEBUG_FLAGS) -CB -no-vec -assume nobuffered_io | ||
| CPPDEF = -Duse_netCDF -Duse_comm_$(CHAN) -DTREAT_OVERLAY -DDEBUG -D__VERBOSE | ||
| MCT_FCFLAGS = $(NCI_REPRO_FLAGS) $(NCI_DEBUG_FLAGS) -ip -fpe0 -xHost |
There was a problem hiding this comment.
should this use -axCORE-AVX2 instead of -xHost so we can run on Broadwell?
There was a problem hiding this comment.
I'm guessing -xHost implies -axCORE-AVX512 if compiled on the Cascade Lake head nodes, which wouldn't run on Broadwell since that only supports -axCORE-AVX2...
There was a problem hiding this comment.
The reason that I had left the flag on that line as -xHost is that I only tested with DEBUG off. I will need to re-test with DEBUG=yes. Also, if DEBUG=yes, then the flags include -no-vec. I will also double check what is done for the other flag files.
There was a problem hiding this comment.
I will also need to check where FFLAGS and MCT_FCFLAGS are used, since the oasis3-mct apparently uses both.
There was a problem hiding this comment.
Addressed by commit a8e4f9b
util/make_dir/config.nci
Outdated
| module load intel-compiler/2019.5.281 | ||
| module load netcdf/4.7.1 | ||
| module load openmpi/4.0.1 | ||
|
|
There was a problem hiding this comment.
these changes to config.nci are unnecessary (but harmless), as config.gadi is used in the gadi-transition branch
There was a problem hiding this comment.
Addressed by commit 861be6d
|
Merge 504aaff uses Discussion for a related merge: COSIMA/libaccessom2@a2e2ef2#r36739504 General discussion: COSIMA/access-om2#184 |
|
This pull request has been mentioned on ACCESS Hive Community Forum. There might be relevant details there: |
Use reproducibility flags to allow Broadwell (Raijin) and Cascade Lake (Gadi) to give bit-identical results. Tested on Gadi.
Closes #13