Skip to content

Conversation

@mathomp4
Copy link
Member

@mathomp4 mathomp4 commented Oct 24, 2025

This is the v12 equivalent of #156 from @biljanaorescanin which adds support for v14 BCs. Per that PR:

This PR does two things:

  1. Adds support for new BCS version v14
  2. Changes path from which we source input for topo files

NOTEs for GCMv12:
At some point we will need to add new test for v14 or change one of the existing ones if that will be our new bcs default.
All tests except f522Toc360 which is ICA are NL3.
Do we need test for mom6 of any kind?

However, it's a bit more complex because, I think, of changes from #144 found in testing.

The issue was I tried to remap from some (saved) MERRA-2 restarts to v14. But in doing so, the end results only had a saltwater_internal_rst file and not the expected split openwater and seaicethermo restarts. So this PR also has changes that says "If my inputs have saltwater then we split and only preserve the openwater and seaicethermo.

Also, in consultation with @biljanaorescanin we remove all references to v13 BCs and make v12 BCs the default until we (eventually) move to v14.

@mathomp4 mathomp4 self-assigned this Oct 24, 2025
@mathomp4 mathomp4 added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Oct 24, 2025
@mathomp4 mathomp4 marked this pull request as ready for review October 27, 2025 14:43
@mathomp4 mathomp4 requested review from a team as code owners October 27, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff The changes in this pull request have verified to be zero-diff with the target branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants