Skip to content

Conversation

@biljanaorescanin
Copy link
Contributor

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?

@biljanaorescanin biljanaorescanin added 0 diff The changes in this pull request have verified to be zero-diff with the target branch. enhancement New feature or request labels Sep 18, 2025
@biljanaorescanin
Copy link
Contributor Author

From testing REMAP package testing was zero diff. But that won't test coupled grids and v14.

Package was also run to remap from GEOS-IT NL3 C180 to v14 c90 M2 to see does new functionality work. All looked good.
Example of linking from remap:
/bin/ln -s /discover/nobackup/projects/gmao/bcs_shared/make_bcs_inputs/atmosphere/TOPO/v1/CF0180x6C/smoothed/topo_DYN_ave_180x1080.data .

/bin/ln -s /discover/nobackup/projects/gmao/bcs_shared/make_bcs_inputs/atmosphere/TOPO/v2/CF0090x6C/smoothed/topo_DYN_ave_90x540.data topo_dynave.data
sbatch -W ./remap_upper.j

I don't expect MOM6 to have issue but @sinakhani can you test coupled grid?

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. enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant