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

Update CIME to ESMCI cime5.8.7-2 #3126

Merged
merged 4,373 commits into from
Aug 16, 2019
Merged

Conversation

jgfouca
Copy link
Member

@jgfouca jgfouca commented Aug 14, 2019

Update CIME to ESMCI cime5.8.7-2

Squash merge of jgfouca/branch-for-to-acme-2019-08-14

Features:

  • Update to latest NCAR PIO2 version
  • improve the config_compilers.xml schema to catch case errors

Bug fixes:

  • make sure output_root path is abspath
  • make sure ~/.cime/config_compilers.xml file is also checked against schema
  • fix indentation bug in get_timing

[BFB]

jedwards4b and others added 30 commits June 5, 2019 09:55
Update modules on cori-haswell and cori-knl

Test suite: scripts_regression_tests.py on cori
Test baseline:
Test namelist changes:
Test status: roundoff

Fixes
User interface changes?:

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

Code review:
Update modules for centos7-linux, fix py3 error in case_run.py

Test suite: scripts_regression_tests.py on centos7-linux and using python 3.6.5, also tested with python 2.7.9
Test baseline:
Test namelist changes:
Test status: roundoff on centos7-linux

Fixes

User interface changes?:

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

Code review:
…-for-mpi-only-runs

Omit OMP_NUM_THREADS in MPI-only runs

Test suite: SMS.f19_g16.X
Test baseline: mpi-only vs. threaded
Test namelist changes: none
Test status: [bit for bit]

Fixes #3082

User interface changes?: none

Update gh-pages html (Y/N)?: N
	modified:   cime/config/e3sm/allactive/config_pesall.xml
Nuopc build fixes

Rework nuopc build to put more logic in Makefile and less in buildexe, i hope this helps make clear
what is happening.

Test suite: scripts_regression_tests.py nuopc testing
Test baseline:
Test namelist changes:
Test status: bit for bit
Fixes

User interface changes?:

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

Code review:
First fix was not py2 compatible, this one is.

Test suite: tested SMS.f09_g16.X by hand on chenne
Test baseline:
Test namelist changes:
Test status: bit for bit
Fixes

User interface changes?:

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

Code review:
Update Spectrum MPI module for PGI, XL and GNU.
Update XL version which has compatible parallel-netcdf install.
Update parallel-netcdf module.
Setting NetCDF paths from env vars defined in machine file.
Previously, OOM errors were encountered with spectrum-mpi module.
That necessitated a flag to use earlier version of romio.
With the MPI update, this is no longer needed.
We use a wrapper script for jsrun options on Summit.
Moving it to stable location under cli115.
Removing csc190 references.
jedwards4b and others added 8 commits August 7, 2019 13:47
…9-08-07

Jgfouca/branch for acme split 2019 08 07

No general changes

Test suite: ./scripts_regression_tests
Test baseline:
Test namelist changes:
Test status: [bit for bit, roundoff, climate changing]

Fixes [CIME Github issue #]

User interface changes?:

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

Code review:
overindented line fixed
Fix an indentation issue

Test suite: by hand
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes this thing which is a comment and not an issue - please don't do this:
773e629?email_source=notifications&email_token=ABOXUGDSWZK3VQ45ULFFEZ3QDMYYDA5CNFSM4IKEFL6KYY3PNVWWK3TUL52HS4DFVVBW63LNNF2EG33NNVSW45FKMNXW23LFNZ2F62LEZYBBAH5S#commitcomment-34611122

User interface changes?:

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

Code review:
…vement

improve the config_compilers.xml schema
Fix indentation problem in compilers.py and make sure that the ~/.cime/config_compilers.xml file is also checked

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

Fixes #3175

User interface changes?:

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

Code review:
update ncpl and cpl_seq_option for mom6
Updates ATM_NCPL, OCN_NCPL, ROF_NCPL, and CPL_SEQ_OPTION for MOM6.

Test suite: aux_mom
Test baseline: /glade/scratch/altuntas/cesm.tests/cheyenne/aux_mom/190808
Test namelist changes: no changes (except mom6 cases)
Test status: : bit for bit (except mom6 cases)

User interface changes?: N

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

Code review:
…branch-for-to-acme-2019-08-14

* commit '1c663d0e30234ff85ec5c205dfbd101dcf62d639': (3218 commits)
  update ncpl and cpl_seq_option for mom6
  overindented line fixed
  improve the config_compilers.xml schema
  async interface not working
  Squashed 'src/externals/pio2/' changes from 6f0afaa48..20457f902
  Specify recursive doubling for MPI_Allreduce on Compy
  make sure output_root path is abspath
  Update CIME to ESMCI cime5.8.7 (#3092)
  Add number of nodes to srun on Anvil
  Use correct mask for ne16pg3 resolution
  Add env var setting for PERL as a work-around while NERSC completes SW maint.
  Add back in the module load of git. When Cori first came back online, this module was missing.
  Updated module versions for Cori (KNL and haswell) after Cori maint.
  clean up
  moving ERS tests to eam_theta
  Update Livermore Computing machine file entries.
  Add default S, M, and L pecounts for oEC60to30v3 G-cases on compy
  Point to new mapping files for r05 <-> f10 mapping
  Disable compiler checks that happen upon cmake launch
  Fix some test fails
  ...
@jgfouca jgfouca added BFB PR leaves answers BFB CIME labels Aug 14, 2019
@jgfouca jgfouca requested a review from rljacob August 14, 2019 20:17
@jgfouca jgfouca self-assigned this Aug 14, 2019
@jgfouca jgfouca changed the title Update CIME to ESMCI cime5.8.7 Update CIME to ESMCI cime5.8.7-2 Aug 15, 2019
jgfouca added a commit that referenced this pull request Aug 15, 2019
Update CIME to ESMCI cime5.8.7-2

Squash merge of jgfouca/branch-for-to-acme-2019-08-14

Features:
* Update to latest NCAR PIO2 version
* improve the config_compilers.xml schema to catch case errors

Bug fixes:
* make sure output_root path is abspath
* make sure ~/.cime/config_compilers.xml file is also checked against schema
* fix indentation bug in get_timing

[BFB]
@jgfouca
Copy link
Member Author

jgfouca commented Aug 15, 2019

Squash merged to next.

@jgfouca jgfouca merged commit fe2b696 into master Aug 16, 2019
@jgfouca jgfouca deleted the jgfouca/branch-for-to-acme-2019-08-14 branch August 16, 2019 17:02
@sarats
Copy link
Member

sarats commented Aug 16, 2019

@jgfouca Does this change timing file outputs? I need to keep track for importing data into PACE

@jgfouca
Copy link
Member Author

jgfouca commented Aug 16, 2019

@sarats , it fixes this bug:
ESMCI/cime#3204

@worleyph
Copy link
Contributor

@jgfouca , what caused this bug? Any idea?

@jgfouca
Copy link
Member Author

jgfouca commented Aug 16, 2019

@worleyph , no idea off the top of my head.

git log -p get_timing.py

^ may reveal the answer.

jgfouca added a commit that referenced this pull request Aug 16, 2019
preview_run: Fix case where mpirun cmd uses env vars

The mpirun cmd was being retrieved before the full environment was
being loaded, leading to unresolved env vars appearing in the command.

[BFB]

* origin/jgfouca/cime/fix_prev_run_env_vars:
  preview_run: Fix case where mpirun cmd uses env vars
  Update CIME to ESMCI cime5.8.7-2 (PR #3126)
jgfouca added a commit that referenced this pull request Aug 16, 2019
preview_run: Fix case where mpirun cmd uses env vars

The mpirun cmd was being retrieved before the full environment was
being loaded, leading to unresolved env vars appearing in the command.

[BFB]

* origin/jgfouca/cime/fix_prev_run_env_vars:
  preview_run: Fix case where mpirun cmd uses env vars
  Update CIME to ESMCI cime5.8.7-2 (PR #3126)
@sarats
Copy link
Member

sarats commented Aug 16, 2019

@worleyph Looks like bug was introduced in ESMCI/cime@773e629
and Az pointed out the problem.

@worleyph
Copy link
Contributor

@sarats and @jgfouca , thanks.

rljacob pushed a commit that referenced this pull request Apr 12, 2021
Update CIME to ESMCI cime5.8.7-2

Squash merge of jgfouca/branch-for-to-acme-2019-08-14

Features:
* Update to latest NCAR PIO2 version
* improve the config_compilers.xml schema to catch case errors

Bug fixes:
* make sure output_root path is abspath
* make sure ~/.cime/config_compilers.xml file is also checked against schema
* fix indentation bug in get_timing

[BFB]
rljacob pushed a commit that referenced this pull request Apr 21, 2021
Nuopc cmeps removed.
Remove the nuopc driver and mediator to a separate repository https://github.com/ESCOMP/CMEPS.git

Test suite: scripts_regression_tests.py
Test baseline:
Test namelist changes:
Test status: bit for bit
Fixes

User interface changes?:

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

Code review:
rljacob pushed a commit that referenced this pull request Apr 21, 2021
Update CIME to ESMCI cime5.8.7-2

Squash merge of jgfouca/branch-for-to-acme-2019-08-14

Features:
* Update to latest NCAR PIO2 version
* improve the config_compilers.xml schema to catch case errors

Bug fixes:
* make sure output_root path is abspath
* make sure ~/.cime/config_compilers.xml file is also checked against schema
* fix indentation bug in get_timing

[BFB]
rljacob pushed a commit that referenced this pull request May 6, 2021
Nuopc cmeps removed.
Remove the nuopc driver and mediator to a separate repository https://github.com/ESCOMP/CMEPS.git

Test suite: scripts_regression_tests.py
Test baseline:
Test namelist changes:
Test status: bit for bit
Fixes

User interface changes?:

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

Code review:
rljacob pushed a commit that referenced this pull request May 6, 2021
Update CIME to ESMCI cime5.8.7-2

Squash merge of jgfouca/branch-for-to-acme-2019-08-14

Features:
* Update to latest NCAR PIO2 version
* improve the config_compilers.xml schema to catch case errors

Bug fixes:
* make sure output_root path is abspath
* make sure ~/.cime/config_compilers.xml file is also checked against schema
* fix indentation bug in get_timing

[BFB]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB CIME
Projects
None yet
Development

Successfully merging this pull request may close these issues.