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

WOMBAT BGC updates #368

Merged
merged 36 commits into from
May 4, 2022
Merged

WOMBAT BGC updates #368

merged 36 commits into from
May 4, 2022

Conversation

aekiss
Copy link
Collaborator

@aekiss aekiss commented Mar 4, 2022

WOMBAT BGC enhancements and bug fixes, as used in 01deg_jra55v140_iaf_cycle4 and the master+bgc branches of the standard ACCESS-OM2 configurations.

This PR supersedes #356

Hakase Hayashida and others added 30 commits February 3, 2021 15:06
- this commit should serve as an example for adding a variable to the coupler.
- in addition to the source code, the new field needs to be added to:
  - namcouple
  - ice/input_ice.nml
  - ocean/input.nml
  in the config directory.
- fix the cpp key definitions.
- fix the order of subroutine so that would work with different models (e.g. ACCESS-CM).
…ation.

compilation successful on my directory.
…not used. with this commit, no need to make unnecessary input files for higher resolutions.
…ing climatology for gas flux calculation. compilation ok but execution bad.
-add MLD-integrated diagnostic following light_limit.
… produce the same output (more or less) but the removed method produces errorneous grid lines.
…ce-to-ocean bgc flux is added to the ocean bgc tendency when computing virtual flux.
@aekiss aekiss mentioned this pull request Mar 4, 2022
@aekiss aekiss requested a review from russfiedler March 15, 2022 22:47
@aekiss
Copy link
Collaborator Author

aekiss commented Mar 15, 2022

Hi @russfiedler would you have the time to look over this PR please? I think we've resolved all the issues now... at least I hope so, as we are using this for IAF cycle 4!

@aekiss
Copy link
Collaborator Author

aekiss commented Apr 12, 2022

Would anyone object if I merge these BGC updates?

f51926f is being used in a production run https://github.com/COSIMA/01deg_jra55_iaf/tree/01deg_jra55v140_iaf_cycle4 and seems to work.

@aekiss aekiss merged commit 730f0bf into master May 4, 2022
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.

None yet

2 participants