Skip to content

Add new burial diagnostics#486

Open
mnlevy1981 wants to merge 3 commits intomarbl-ecosys:developmentfrom
mnlevy1981:bury_diags
Open

Add new burial diagnostics#486
mnlevy1981 wants to merge 3 commits intomarbl-ecosys:developmentfrom
mnlevy1981:bury_diags

Conversation

@mnlevy1981
Copy link
Collaborator

Three derivatives used to compute burial coefficients when ladjust_bury_coeff = .true. are now available as diagnostics regardless of the value of ladjust_bury_coeff. They are not included in the default diagnostic set but will be useful for computing the burial coefficients offline for GCMs that do not provide the global operators necessary to adjust the burial coefficients on the fly.

Always compute the derivative of the burial for POC, POP, and bSi with respect
to that particle's burial coefficient; we want to make it available in
diagnostic output regardless of value of ladjust_bury_coeff
d_POC_bury_d_bury_coeff, d_POC_bury_d_bury_coeff, and d_POC_bury_d_bury_coeff
are now available diagnostics, stored in store_diagnostics_particulates()
Default is to not include the three new variables, but they are available for
users to turn on if desired
@mnlevy1981 mnlevy1981 requested a review from klindsay28 November 6, 2025 16:01
@mnlevy1981
Copy link
Collaborator Author

I expect some regression test failures because these new diagnostics are not included in the baselines for the call_compute_subroutines tests; I will create new baselines after the PR is reviewed and we are happy with the changes, and will not merge this until the CI tests all pass.

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 this pull request may close these issues.

1 participant