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

initialize cpl_scalar field when created #83

Merged

Conversation

DeniseWorthen
Copy link
Collaborator

Fixes ufs-community/ufs-weather-model#2369.

See related UFS #2369 for details.

@zach1221
Copy link

zach1221 commented Aug 5, 2024

@DeniseWorthen testing is complete on your UFS-WM PR 2370. Feel free to merge this CICE sub-component PR.

@DeniseWorthen DeniseWorthen merged commit 2ffee5c into NOAA-EMC:emc/develop Aug 5, 2024
1 check passed
NickSzapiro-NOAA pushed a commit to NickSzapiro-NOAA/CICE that referenced this pull request Aug 9, 2024
NickSzapiro-NOAA added a commit to NickSzapiro-NOAA/CICE that referenced this pull request Aug 21, 2024
…e cpl_scalar field when created for UFS (CICE-Consortium#969)


These are two commits cherry-picked from as in UFS and needed to close NOAA-EMC#84.

This PR adds the ability for CICE to write restart files at the end of the run (independent of other settings) and controlled via the CMEPS configuration option write_restart_at_endofrun. Setting this configuration option to True creates a restart file in the same way for CMEPS, MOM6, and CICE.

This PR also initializes the scalar field value for the index for NTile (implemented for FV3) not used or set in CICE. In certain cases, the scalar field value for this index has been found to be non-zero (NaN in debug compiles). This is the cause of the failure reported in ufs-community/ufs-weather-model#2338.

* Add end of run functionality to CICE (NOAA-EMC#77)
* initialize cpl_scalar field when created (NOAA-EMC#83)

---------

Co-authored-by: Daniel Sarmiento <42810219+dpsarmie@users.noreply.github.com>
Co-authored-by: Denise Worthen <denise.worthen@noaa.gov>
NickSzapiro-NOAA added a commit that referenced this pull request Sep 25, 2024
…e cpl_scalar field when created for UFS (CICE-Consortium#969)


These are two commits cherry-picked from as in UFS and needed to close #84.

This PR adds the ability for CICE to write restart files at the end of the run (independent of other settings) and controlled via the CMEPS configuration option write_restart_at_endofrun. Setting this configuration option to True creates a restart file in the same way for CMEPS, MOM6, and CICE.

This PR also initializes the scalar field value for the index for NTile (implemented for FV3) not used or set in CICE. In certain cases, the scalar field value for this index has been found to be non-zero (NaN in debug compiles). This is the cause of the failure reported in ufs-community/ufs-weather-model#2338.

* Add end of run functionality to CICE (#77)
* initialize cpl_scalar field when created (#83)

---------

Co-authored-by: Daniel Sarmiento <42810219+dpsarmie@users.noreply.github.com>
Co-authored-by: Denise Worthen <denise.worthen@noaa.gov>
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.

cpl_scalar fields should be initialized in non-CSG components when created
4 participants