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

added felim to diazo iron limitation #31

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

charliestock
Copy link
Contributor

This change updated the diazotroph iron limitation to be consistent with a change in the iron limitation calculation made for the other phytoplankton types in COBALTv3.0. The level of iron limitation is now calculated as the maximum of the limitations imposed by the internal cell quota (def_fe) and the ambient iron uptake limitation (felim).

While this fix changes answers, the effect is very small. It mainly removes small regions near river mouth where iron ends up being the most limiting nutrient simply because it takes a short time for high internal iron quotas to be established in high iron environments. Using the maximum of def_fe and felim removes this delay and puts iron on equal footing with N and P.

This adjustment also only effects the diazotrophs, which are generally a small fraction of the phytoplankton community.

@yichengt900
Copy link
Collaborator

Thanks, @charliestock. It looks good to me. As you pointed out, the effect of this change is probably small and was not even caught by our 1D test case (which makes sense because our 1D test case is located in the middle of the ocean). I will approve it for now and will merge this PR once it has passed the NWA12 RT test.

@yichengt900
Copy link
Collaborator

Ok it also passed the NWA12 RT test at the CEFI-regional-mom6 side. Will merge this PR now.

@yichengt900 yichengt900 merged commit d4e20e4 into dev/cefi Apr 4, 2024
1 check passed
@yichengt900 yichengt900 deleted the addfelimdiazo/clean_code branch April 4, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants