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 mpas-source submodule for ocean halo reuse #2387

Merged
merged 1 commit into from
Jun 12, 2018

Conversation

mark-petersen
Copy link
Contributor

Update mpas-source submodule for ocean halo reuse

@mark-petersen mark-petersen requested a review from jonbob June 7, 2018 14:08
@mark-petersen
Copy link
Contributor Author

@jonbob there is something wrong with

components/clm/src/external_models/fates

see #2386. But the mpas-source update here is correct.

Copy link
Member

@rljacob rljacob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes to FATES have to be removed.

@mark-petersen mark-petersen force-pushed the mark-petersen/ocean/halo_reuse branch 2 times, most recently from aea1ec1 to 0537467 Compare June 8, 2018 19:36
@mark-petersen
Copy link
Contributor Author

OK, I updated so only mpas-source is altered. Tested with:

./create_test ERS_Ld3.T62_oQU240.GMPAS-IAF.cori-knl_intel
./create_test SMS_D.T62_oQU120_ais20.MPAS_LISIO_TEST.cori-knl_gnu

@mark-petersen
Copy link
Contributor Author

@philipwjones and @Larofeticus FYI here is the E3SM PR. Could you please link to or describe the past testing that you've done. @jonbob will do a few standard tests on this PR as well, when he is back next week.

@mark-petersen mark-petersen force-pushed the mark-petersen/ocean/halo_reuse branch from 0537467 to 64ac97e Compare June 8, 2018 20:57
@Larofeticus
Copy link

The recorded history of this is.

Adding three routines to closed repository MPAS develop which facilitate reusing halo exchange data structures throughout a simulation.
MPAS-Dev/MPAS#1453

Using those routines in the ocean core revealed a memory leak.
MPAS-Dev/MPAS#1458

Memory leak found and corrected in framework.
MPAS-Dev/MPAS#1515

Ocean core modification to reuse halo data structures in the open repository:
MPAS-Dev/MPAS-Model#41

@jonbob
Copy link
Contributor

jonbob commented Jun 11, 2018

I ran two identical T62_oECv3.GMPAS-IAF.cori-knl_intel tests, one with and one without this mpas update. After one month, the results were bfb and the ocean performance increased slightly from 2.16 to 2.22 sypd. I saw no sign of a memory leak either, so I think this PR can be satisfied.

@jonbob jonbob added the BFB PR leaves answers BFB label Jun 11, 2018
jonbob added a commit that referenced this pull request Jun 11, 2018
This PR updates the mpas-source submodule for ocean halo reuse

Tested with:
* ERS_Ld3.T62_oQU240.GMPAS-IAF.cori-knl_intel
* SMS_D.T62_oQU120_ais20.MPAS_LISIO_TEST.cori-knl_gnu
* T62_oECv3.GMPAS-IAF.cori-knl_intel one-month runs, BFB w/ and
  w/o this update

[BFB]
@jonbob
Copy link
Contributor

jonbob commented Jun 11, 2018

merged to next

@jonbob jonbob merged commit 64ac97e into master Jun 12, 2018
jonbob added a commit that referenced this pull request Jun 12, 2018
This PR updates the mpas-source submodule for ocean halo reuse

Tested with:
* ERS_Ld3.T62_oQU240.GMPAS-IAF.cori-knl_intel
* SMS_D.T62_oQU120_ais20.MPAS_LISIO_TEST.cori-knl_gnu
* T62_oECv3.GMPAS-IAF.cori-knl_intel one-month runs, BFB w/ and
  w/o this update

[BFB]
@jonbob
Copy link
Contributor

jonbob commented Jun 12, 2018

merged to master

@jonbob jonbob deleted the mark-petersen/ocean/halo_reuse branch June 12, 2018 15:10
jgfouca added a commit that referenced this pull request Jun 28, 2018
Deccsmize

Remove text 'ccsm' and 'cesm' and replace appropriately
Test suite: scripts_regression_tests.py
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes #2387

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
Labels
BFB PR leaves answers BFB enhancement mpas-ocean
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants