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

Changing clobber mode in ocean conservation analysis #6258

Merged

Conversation

darincomeau
Copy link
Member

Previously the clobber mode in the ocean conservation analysis member was set to 'append', which would throw errors to log.ocean.*.err files when rerunning a case:

ERROR: Writing to stream 'conservationCheckOutput' would overwrite record    1 in file 'SMS_Lm1_P512.ne30pg2_r05_IcoswISC30E3r5.CRYO1850.chrysalis_intel.20240225_133103_09nrwo.mpaso.hist.am.conservationCheck.0001-02-01.nc',
ERROR:     but clobber_mode is set to 'append'.

Changing clobber mode to truncate avoids these errors.

[BFB]

@darincomeau darincomeau added mpas-ocean BFB PR leaves answers BFB labels Feb 25, 2024
@darincomeau darincomeau requested a review from xylar February 25, 2024 22:39
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.

Approving based on an inspection of the code and @darincomeau's testing. I ran into the same error messages that he mentions above in my previous re-runs when the conservation check analysis member is turned on.

jonbob added a commit that referenced this pull request Mar 5, 2024
…(PR #6258)

Changing clobber mode in ocean conservation analysis

Previously the clobber mode in the ocean conservation analysis member
was set to 'append', which would throw errors to log.ocean.*.err files
when rerunning a case.

Changing clobber mode to truncate avoids these errors.

[BFB]
@jonbob
Copy link
Contributor

jonbob commented Mar 5, 2024

passes:

  • ERP_Ld3.ne30pg2_r05_IcoswISC30E3r5.WCYCL1850.chrysalis_intel.allactive-pioroot1
  • ERS_Ld5.T62_oQU120.CMPASO-NYF.chrysalis_intel

merged to next

@jonbob jonbob merged commit ff02130 into E3SM-Project:master Mar 6, 2024
10 checks passed
@jonbob
Copy link
Contributor

jonbob commented Mar 6, 2024

merged to master

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants