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.

Sign up for GitHub

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

Jump to bottom

CM2G63L is not consistent with production ESM2G #42

Closed
adcroft opened this issue Aug 26, 2014 · 3 comments
Closed

CM2G63L is not consistent with production ESM2G #42

adcroft opened this issue Aug 26, 2014 · 3 comments
Assignees
@adcroft
Labels
enhancement

Comments

@adcroft
Copy link
Collaborator

adcroft commented Aug 26, 2014

It appears that CM2G63L is not consistent with the actual ocean configuration used in ESM2G. The following overrides were always used when running ESM2G:

#override VELOCITY_TOLERANCE = 3.0e8
#override CONTINUITY_CFL_LIMIT = 1.0
#override CONT_PPM_BETTER_ITER = False
#override CONT_PPM_USE_VISC_REM_MAX = False
#override USE_BT_CONT_TYPE = False
#override RESCALE_BT_FACE_AREAS = True
#override BT_STRONG_DRAG = False
#override NONLINEAR_BT_CONTINUITY = True

CM2G63L should be updated to reflect ESM2G and new timestats generated.
@jkrasting
Copy link
Contributor

Do we also want to change the example to initialize from
WOA05_pottemp_salt.nc rather than the GOLD_IC.2010.11.15.nc file?

Dr. John Krasting
Physical Scientist (NOAA Federal)
NOAA/Geophysical Fluid Dynamics Laboratory
Biogeochemistry, Ecosystems, and Climate Group
Princeton University Forrestal Campus
201 Forrestal Road
Princeton, NJ 08540

P. (609) 452-5359
F. (609) 987-5063

On Tue, Aug 26, 2014 at 12:54 PM, Alistair Adcroft (GFDL) <
[email protected]> wrote:

It appears that CM2G63L is not consistent with the actual ocean
configuration used in ESM2G. The following overrides were always used when
running ESM2G:

#override VELOCITY_TOLERANCE = 3.0e8
#override CONTINUITY_CFL_LIMIT = 1.0
#override CONT_PPM_BETTER_ITER = False
#override CONT_PPM_USE_VISC_REM_MAX = False
#override USE_BT_CONT_TYPE = False
#override RESCALE_BT_FACE_AREAS = True
#override BT_STRONG_DRAG = False
#override NONLINEAR_BT_CONTINUITY = True

CM2G63L should be updated to reflect ESM2G and new timestats generated.

—
Reply to this email directly or view it on GitHub
#42.

@MJHarrison-GFDL
Copy link
Contributor

Good idea!

On Wed, Aug 27, 2014 at 9:03 AM, John Krasting [email protected]
wrote:

Do we also want to change the example to initialize from
WOA05_pottemp_salt.nc rather than the GOLD_IC.2010.11.15.nc file?

Dr. John Krasting
Physical Scientist (NOAA Federal)
NOAA/Geophysical Fluid Dynamics Laboratory
Biogeochemistry, Ecosystems, and Climate Group
Princeton University Forrestal Campus
201 Forrestal Road
Princeton, NJ 08540

P. (609) 452-5359
F. (609) 987-5063

On Tue, Aug 26, 2014 at 12:54 PM, Alistair Adcroft (GFDL) <
[email protected]> wrote:

It appears that CM2G63L is not consistent with the actual ocean
configuration used in ESM2G. The following overrides were always used
when
running ESM2G:

#override VELOCITY_TOLERANCE = 3.0e8
#override CONTINUITY_CFL_LIMIT = 1.0
#override CONT_PPM_BETTER_ITER = False
#override CONT_PPM_USE_VISC_REM_MAX = False
#override USE_BT_CONT_TYPE = False
#override RESCALE_BT_FACE_AREAS = True
#override BT_STRONG_DRAG = False
#override NONLINEAR_BT_CONTINUITY = True

CM2G63L should be updated to reflect ESM2G and new timestats generated.

—
Reply to this email directly or view it on GitHub
#42.

—
Reply to this email directly or view it on GitHub
#42 (comment)
.

@adcroft adcroft self-assigned this Aug 27, 2014
adcroft added a commit that referenced this issue Aug 27, 2014
@adcroft
Minor cleanup of CM2G63L/MOM_input
100d4ec 
- This just reconciles apparent differences between MOM_input and
  MOM_parameter_doc.short for CM2G63L (in preparation for addressing #42).
- No actual parameter changes occurred.
- No answer changes.
adcroft added a commit that referenced this issue Aug 27, 2014
@adcroft
*Reconfigured CM2G63L to reflect actual ESM2G
acf3628 
- As raised in issue #42, the parameters recorded in examples/coupled_AM2_SIS/CM2G63L
  were not exactly those used in the production runs of ESM2G. They had meant to have
  been.
- Parameter changes can be seen in examples/coupled_AM2_SIS/CM2G63L/MOM_parameter_doc.all
  and were baed on the MOM_override being used for ESM2G:
     #override VELOCITY_TOLERANCE = 3.0e8
     #override CONTINUITY_CFL_LIMIT = 1.0
     #override CONT_PPM_BETTER_ITER = False
     #override CONT_PPM_USE_VISC_REM_MAX = False
     #override USE_BT_CONT_TYPE = False
     #override RESCALE_BT_FACE_AREAS = True
     #override BT_STRONG_DRAG = False
     #override NONLINEAR_BT_CONTINUITY = True
- We should no longer need any physical flags/parameters in MOM_override to run an ESM2G.
- This is an answer change for this experiment only.
- This closes issue #42.
@jkrasting
Copy link
Contributor

Thanks ... would it be possible to make a dev/master/ tag that
includes d808041? Just trying to keep the XMLs and directory structures
clean.

Dr. John Krasting
Physical Scientist (NOAA Federal)
NOAA/Geophysical Fluid Dynamics Laboratory
Biogeochemistry, Ecosystems, and Climate Group
Princeton University Forrestal Campus
201 Forrestal Road
Princeton, NJ 08540

P. (609) 452-5359
F. (609) 987-5063

On Wed, Aug 27, 2014 at 4:06 PM, Alistair Adcroft (GFDL) <
[email protected]> wrote:

Closed #42 #42 via
d808041
d808041
.

—
Reply to this email directly or view it on GitHub
#42 (comment).

@adcroft adcroft mentioned this issue Sep 6, 2014
Closed
nikizadehgfdl pushed a commit to nikizadehgfdl/MOM6 that referenced this issue Oct 9, 2017
@adcroft
SIS2: Merge branch 'user/nnz/fix_restart_issue42' into dev/master
b1fda6d 
- NOAA-GFDL/SIS2@44382d6 Merge branch 'user/nnz/fix_restart_issue42' into dev/master
- NOAA-GFDL/SIS2@39eb2ff Merge branch 'dev/master' into user/nnz/fix_restart_issue42
- NOAA-GFDL/SIS2@23d7cb6 Merge branch 'user/nnz/fix_openmp_issues_20160829' into dev/master
- NOAA-GFDL/SIS2@81ba1ed Fix restart issue mom-ocean#42
- NOAA-GFDL/SIS2@503bfe2 Fix compilation issues with -openmp
gustavo-marques pushed a commit to gustavo-marques/MOM6 that referenced this issue Apr 16, 2018
@alperaltuntas
Merge pull request mom-ocean#42 from gustavo-marques/merge-dev-master…
8095e8f 
…-2018-02-28

Merging latest dev/master Feb 28 2018
MJHarrison-GFDL pushed a commit to MJHarrison-GFDL/MOM6 that referenced this issue Jun 15, 2021
@nikizadehgfdl
Fix restart issue mom-ocean#42
81ba1ed 
- Fixes restart mismatch issue mom-ocean#42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adcroft adcroft

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@adcroft @jkrasting @MJHarrison-GFDL