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

Add spinup diagnostic presets to historical+concentrations configuration #106

Merged

Conversation

blimlim
Copy link

@blimlim blimlim commented Dec 1, 2024

Closes the historical half of #104.

This pull request adds a new "spinup" output preset for each model component. These request a reduced amount of output, tailored to long spinup simulations, producing ~ 900M output per year.

The changes relative to the "standard" preset are listed in the following spreadsheet:
spinup_preset (1).xls

Four new variables were also added to all three presets:

  • temp_yflux_adv_int_z
  • temp_xflux_adv_int_z
  • salt_yflux_adv_int_z
  • salt_xflux_adv_int_z

as discussion identified them as useful.

@blimlim blimlim requested a review from MartinDix December 3, 2024 05:45
@MartinDix
Copy link

I'd like to keep sub-surface runoff because it's necessary for calculating a water budget.

@blimlim
Copy link
Author

blimlim commented Dec 4, 2024

I'd like to keep sub-surface runoff because it's necessary for calculating a water budget.

Good idea! I've added this back in (8235), and also added back "geopotential height on plev grid" (30207) as suggested on the Hive Forum

Copy link

@MartinDix MartinDix left a comment

Choose a reason for hiding this comment

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

Happy with this now

@blimlim blimlim merged commit 53da2fe into dev-historical+concentrations Dec 11, 2024
10 checks passed
@blimlim blimlim deleted the 104-add-spinup-presets-historical branch December 11, 2024 03:25
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.

2 participants