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

Updates for the SOCA config for updating pinned JEDI build #430

Merged
merged 4 commits into from
Sep 26, 2024

Conversation

Dooruk
Copy link
Collaborator

@Dooruk Dooruk commented Sep 18, 2024

Following some SABER updates, configuration for SOCA needs to be updated to use JEDI builds past Aug 3rd.

Explicit_diffusion became generic (i.e., it can be shared with other models). For SOCA, it has to use the model interface and that introduces some quirks. The correlation scales need to be named as ocean model variables (instead of hz or vt, tocn and ssh). And the state_variable in background_diffusion_vt.yaml has to only contain the correlation scale variable (tocn in this case). Hopefully this will be fixed in a future update.

Dependencies:

@rtodling tested with an August 31st JEDI build and results are practically identical.

  • Requires Static file changes which are currently located in my local folder.

@Dooruk Dooruk changed the title Updates for the SOCA config Updates for the SOCA config for updating pinned JEDI build Sep 18, 2024
Copy link

@DavidSRussell DavidSRussell left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@Dooruk Dooruk merged commit 5be74e9 into develop Sep 26, 2024
28 of 43 checks passed
@Dooruk Dooruk deleted the feature/generic_saber branch September 26, 2024 15:48
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