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

33-a update cice diagnostics for historical configuration #35

Closed
wants to merge 4 commits into from

Conversation

blimlim
Copy link

@blimlim blimlim commented Jul 31, 2024

This pull request completes the portion of #33 for the historical configuration. It:

  • Removes daily output
  • Removes diagnostics impacted by the ESM1.5 coupling bug, and adds a comment with brief explanation.

@blimlim blimlim requested a review from anton-seaice July 31, 2024 06:05
, f_aice = 'dm'
, f_uvel = 'dm'
, f_vvel = 'dm'
, f_hi = 'm'

Choose a reason for hiding this comment

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

why are we turning off daily output ? Are there minutes or something where this is documented?

Choose a reason for hiding this comment

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

if we are doing this to save space, maybe turning of aicen, vicen and other variables with thickness categories makes sense.

Copy link
Author

Choose a reason for hiding this comment

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

Great question. Do you know if these variables are analysed very often? If they're useful enough we could keep them in, and then remove them from the "reduced/paleo" preset.

Actually getting back to the presets, I just realised it probably makes sense to set all the ice history requests as part of putting together the three presets (detailed, standard, and reduced), rather setting some of them now and redoing it later. Sorry about this! Would it be alright if I were to delete the two PRs relating to #33, and then we can set all the ice requests when we put together the three presets?

@blimlim
Copy link
Author

blimlim commented Aug 14, 2024

Closing this PR as selecting CICE diagnostics will be part of setting up the output presets.

@blimlim blimlim closed this Aug 14, 2024
@blimlim blimlim deleted the 33-historical-update-cice-diagnostics branch August 26, 2024 23:57
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