Icepack interface for binary restarts

[eclare108213]

Added Icepack interface calls to binary restart module and isolated netcdf bgc code.

• Developer(s): E. Hunke

• Please suggest code Pull Request reviewers in the column at right.

• Are the code changes bit for bit, different at roundoff level, or more substantial? BFB

• Is the documentation being updated with this PR? (Y/N) N
  If not, does the documentation need to be updated separately at a later time? (Y/N) N
  Note: "Documentation" includes information on the wiki and .rst files in doc/source/,
  which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/.

• Other Relevant Details:

The modal_aero option requires netcdf input, so I added an abort if modal_aero=T and netcdf is not available. @njeffery is it possible to include this data directly in the code or provide binary for it?

The change to io/netcdf/ice_restart.F90 is cosmetic.

This PR is not yet fully tested, just trying to keep Rob going...

[eclare108213]

Smoke and restart tests using binary I/O work. These options must be set in ice_in to run them:

ice_ic         = 'default' or 'none'
restart        = .false.

Ran the base_suite; results are at https://github.com/CICE-Consortium/Test-Results/wiki/e203dcbf71.pinto.intel.180608.172808

pinto_intel_restart_gx1_40x4_debug_droundrobin failures are because this case asks for 10 nodes, but the maximum number of nodes that can be requested is 5, for the standard qos on pinto. This will always fail on pinto. The other failure is for a box configuration, which still needs to be debugged.

This PR should be ready to merge.

[njeffery]

@eclare: Yes. I can create binary versions.

…

________________________________ From: Elizabeth Hunke <notifications@github.com> Sent: Thursday, June 7, 2018 4:20:16 PM To: CICE-Consortium/CICE Cc: Jeffery, Nicole; Mention Subject: [CICE-Consortium/CICE] Icepack interface for binary restarts (#148) Added Icepack interface calls to binary restart module and isolated netcdf bgc code. * Developer(s): E. Hunke * Please suggest code Pull Request reviewers in the column at right. * Are the code changes bit for bit, different at roundoff level, or more substantial? BFB * Is the documentation being updated with this PR? (Y/N) N If not, does the documentation need to be updated separately at a later time? (Y/N) N Note: "Documentation" includes information on the wiki and .rst files in doc/source/, which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/. * Other Relevant Details: The modal_aero option requires netcdf input, so I added an abort if modal_aero=T and netcdf is not available. @njeffery<https://github.com/njeffery> is it possible to include this data directly in the code or provide binary for it? The change to io/netcdf/ice_restart.F90 is cosmetic. This PR is not yet fully tested, just trying to keep Rob going...

________________________________ You can view, comment on, or merge this pull request online at: #148 Commit Summary * Icepack interface for binary restarts File Changes * M cicecore/cicedynB/general/ice_forcing_bgc.F90<https://github.com/CICE-Consortium/CICE/pull/148/files#diff-0> (6) * M cicecore/cicedynB/infrastructure/io/io_binary/ice_restart.F90<https://github.com/CICE-Consortium/CICE/pull/148/files#diff-1> (78) * M cicecore/cicedynB/infrastructure/io/io_netcdf/ice_restart.F90<https://github.com/CICE-Consortium/CICE/pull/148/files#diff-2> (8) Patch Links: * https://github.com/CICE-Consortium/CICE/pull/148.patch * https://github.com/CICE-Consortium/CICE/pull/148.diff — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#148>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AH1K6tZFH8Uq6W3v7Tf5Pj3S0VsM1H3aks5t6acggaJpZM4UfNg5>.