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

MOSART negative channel storage stopped v3.LR simulation #6622

Closed
wlin7 opened this issue Sep 17, 2024 · 0 comments · Fixed by #6623
Closed

MOSART negative channel storage stopped v3.LR simulation #6622

wlin7 opened this issue Sep 17, 2024 · 0 comments · Fixed by #6623

Comments

@wlin7
Copy link
Contributor

wlin7 commented Sep 17, 2024

The master code base that includes the changes via PR #6313, can cause v3.LR.piControl simulation to stop running due to negative channel storage that is no more tolerated. This was not an issue for the original v3.LR.piControl simulation with v3.0.0 tag.

Similar issue was also reported in the comment section of #6313.

For debugging, a branch run based off year 0416-01-01 of v3.LR.piControl can reproduce the issue in 8 months (occur on 0416-08-25), using master branch that has #6313 in place (make sure not being reverted)

Since its occurrence is unpredictable, and v3.LR simulation campaigns still have a high number of long simulations (single forcings over historical period, SSPs) to be completed with commits in the latest master branch (until v3.0.1 is tagged), we may have to revert #6313. The revert can then be reverted after v3.0.1 tag is created. And after that, a physics-based fix will still need to be implemented to either prevent the negative storage from occurring, apply a fixer, or make it more tolerant.

@peterdschwartz @hydrotian , please weigh in.

wlin7 added a commit that referenced this issue Sep 17, 2024
…#6623)

Revert PR #6313 aborting run on negative channel storage

The enforced abort whenever MOSART has negative channel storage
may prevent v3.LR simulations from finishing, if the code base that includes
to ensure new v3.LR simulations that need to use the latest commits to master
are free from this issue.

Fixes #6622.

[BFB]
@wlin7 wlin7 closed this as completed in 81312e9 Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant