rpointer fix for cesm - Githubissues

jedwards4b commented 3 weeks ago

For detailed information about submitting Pull Requests (PRs) to the CICE-Consortium, please refer to: https://github.com/CICE-Consortium/About-Us/wiki/Resource-Index#information-for-developers

PR checklist

[X] Short (1 sentence) summary of your PR: add status feature to open existing file, remove no longer needed cesm pointer_file modification
[X] Developer(s): Jim Edwards (NCAR)
[X] Suggest PR reviewers from list in the column to the right. Dave Bailey
[X] Please copy the PR test results link or provide a summary of testing completed below.
How much do the PR code changes differ from the unmodified code?
- [X] bit for bit
- [ ] different at roundoff level
- [ ] more substantial
Does this PR create or have dependencies on Icepack or any other models?
- [ ] Yes
- [X] No
Does this PR update the Icepack submodule? If so, the Icepack submodule must point to a hash on Icepack's main branch.
- [ ] Yes
- [X] No
Does this PR add any new test cases?
- [ ] Yes
- [X] No
Is the documentation being updated? ("Documentation" includes information on the wiki or in the .rst files from doc/source/, which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/. A test build of the technical docs will be performed as part of the PR testing.)
- [ ] Yes
- [X] No, does the documentation need to be updated at a later time?
  - [ ] Yes
  - [X] No
[X] Please document the changes in detail, including why the changes are made. This will become part of the PR commit log. With the change in cesm3_cice6_5_0_9 the correct cesm pointer_file name is written to the namelist file and so it is no longer required to append the instance number in a multi instance case.

dabail10 commented 3 weeks ago

I believe everything here is in CESMCOUPLED ifdefs and so should not be added to io_netcdf or io_pio.

anton-seaice commented 3 weeks ago