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

Update marbl iron forcing #231

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mnlevy1981
Copy link
Collaborator

Once marbl-ecosys/MARBL#464 and NCAR/MOM6#340 are merged, we will need to update the git externals in this repository to use the latest tags of both. Other changes in this PR:

  1. The MARBL PR adds the ability to run with variable N:C, which adds N-tracers to every autotroph functional type; this requires an updated initial condition file that contains these new tracers
  2. The new iron forcing introduces a new reduced sediment flux field that needs to be provided to MARBL, and also modifies what is expected from the existing sediment flux field... so both of those files were updated.

This will require updated MARBL and MOM6 tags as well
Also updated the ecosys IC file for 3p1z (since that could potentially be run
with variable N:C)
@mnlevy1981
Copy link
Collaborator Author

took a couple tries, but finally passing CI :) (first the check_inputdata test reminded me to update input_data_list.yaml, and then it pointed out that I had only imported two of the three new input files into the inputdata repo)

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