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

Set default config_check_ssh_consistency for oQU240wLI to false #6746

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

jonbob
Copy link
Contributor

@jonbob jonbob commented Nov 12, 2024

Change the default config_check_ssh_consistency setting for oQU240wLI to false, to avoid confusing warnings/errors from tests with this grid. Otherwise we end up with log.ocean.err files complaining about:
"ERROR: Warning: Sea surface height outside of acceptable range (20m)."
In general, mpaso sets config_check_ssh_consistency to false for grids with ice shelf cavities, but this one was missed.

[NML] for configurations with oQU240wLI
[BFB]

@jonbob jonbob added mpas-ocean BFB PR leaves answers BFB NML labels Nov 12, 2024
@jonbob jonbob requested a review from xylar November 12, 2024 23:01
@jonbob jonbob self-assigned this Nov 12, 2024
@jonbob
Copy link
Contributor Author

jonbob commented Nov 12, 2024

Testing with ERS_Ld5.TL319_oQU240wLI_ais8to30.MPAS_LISIO_JRA1p5.chrysalis_intel.mpaso-ocn_glcshelf shows that the results are not changed by this PR, but the log.ocean.XXXX.err files are no longer produced

Copy link

PR Preview Action v1.4.8
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6746/
on branch gh-pages at 2024-11-12 23:03 UTC

Copy link
Contributor

@xylar xylar left a comment

Choose a reason for hiding this comment

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

@jonbob, yes, absolutely! This looks good. Thanks for making this much-needed change.

jonbob added a commit that referenced this pull request Nov 13, 2024
Set default config_check_ssh_consistency for oQU240wLI to false

Change the default config_check_ssh_consistency setting for oQU240wLI to
false, to avoid confusing warnings/errors from tests with this grid.
Otherwise we end up with log.ocean.err files complaining about:
   ERROR: Warning: Sea surface height outside of acceptable range (20m).
In general, mpaso sets config_check_ssh_consistency to false for grids
with ice shelf cavities, but this one was missed.

[NML] for configurations with oQU240wLI
[BFB]
@jonbob
Copy link
Contributor Author

jonbob commented Nov 13, 2024

Passes:

  • ERP_Ld3.ne30pg2_r05_IcoswISC30E3r5.WCYCL1850.chrysalis_intel.allactive-pioroot1
  • ERS_Ld5.TL319_oQU240wLI_ais8to30.MPAS_LISIO_JRA1p5.chrysalis_intel.mpaso-ocn_glcshelf (with only an NML DIFF)

merged to next

@jonbob jonbob merged commit 741864b into master Nov 14, 2024
12 checks passed
@jonbob jonbob deleted the jonbob/ocn/qu240wli-ssh-check branch November 14, 2024 17:04
@jonbob
Copy link
Contributor Author

jonbob commented Nov 14, 2024

merged to master and expected NML DIFFs blessed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB mpas-ocean NML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants