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

Increase value of sediment POC degradation constant #464

Merged

Conversation

JorgSchwinger
Copy link
Contributor

@JorgSchwinger JorgSchwinger commented Jan 16, 2025

This increases the default value of disso_poc quite substantially. A larger value for this constant leads to a number of improvements (higher PP, reduced deep sea oxygen biases). Even if this parameter will need fine-tuning, the new value has a much better order of magnitude.

closes #461

@jmaerz
Copy link
Collaborator

jmaerz commented Jan 16, 2025

Hi @JorgSchwinger , just to be sure (and to document): the new value is for the default model (so not for the recently included T- and O2-dependent sediment remin #454), right?

Copy link
Collaborator

@jmaerz jmaerz left a comment

Choose a reason for hiding this comment

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

Thanks for performing runs and for putting this in!

@JorgSchwinger
Copy link
Contributor Author

It is ok-ish for both, the T-dependent remineralization does not have such a huge impact (globally) as far as I have seen. As mentioned above, this is not yet fine-tuned, but the order of magnitude is now much better.

@jmaerz
Copy link
Collaborator

jmaerz commented Jan 16, 2025

Hi @JorgSchwinger , thanks for checking - yes, the value is generally closer to what I would expect (and to which I tuned the sediment thus far for the N-cycle).

@jmaerz jmaerz added this to the release v1.7.0 tag milestone Jan 16, 2025
@jmaerz jmaerz mentioned this pull request Jan 16, 2025
17 tasks
@TomasTorsvik TomasTorsvik self-assigned this Jan 17, 2025
@TomasTorsvik TomasTorsvik added the iHAMOCC Issue mainly concerns the iHAMOCC code base label Jan 17, 2025
@TomasTorsvik TomasTorsvik merged commit 3373bbc into NorESMhub:master Jan 17, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iHAMOCC Issue mainly concerns the iHAMOCC code base
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Correct POC remineralization rate
3 participants