Skip to content

nuopc/cmeps cap fixes; remove compiler warning and fix missing j-loop index #944

Merged
apcraig merged 1 commit into
CICE-Consortium:mainfrom
DeniseWorthen:feature/emccapfixes
Apr 15, 2024
Merged

nuopc/cmeps cap fixes; remove compiler warning and fix missing j-loop index #944
apcraig merged 1 commit into
CICE-Consortium:mainfrom
DeniseWorthen:feature/emccapfixes

Conversation

@DeniseWorthen
Copy link
Copy Markdown
Contributor

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

  • Short (1 sentence) summary of your PR:
    Fix two small issues in nuopc/cmeps cap: fix to resolve compiler warning (operational req) and fix to add missing j-loop
  • Developer(s):
    @DeniseWorthen
  • Suggest PR reviewers from list in the column to the right.
  • Please copy the PR test results link or provide a summary of testing completed below.
    ENTER INFORMATION HERE
  • How much do the PR code changes differ from the unmodified code?
    • bit for bit
    • different at roundoff level
    • more substantial
  • Does this PR create or have dependencies on Icepack or any other models?
    • Yes
    • No
  • Does this PR update the Icepack submodule? If so, the Icepack submodule must point to a hash on Icepack's main branch.
    • Yes
    • No
  • Does this PR add any new test cases?
    • Yes
    • 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
    • No, does the documentation need to be updated at a later time?
      • Yes
      • No
  • Please document the changes in detail, including why the changes are made. This will become part of the PR commit log.

Resolve compiler warning arising from ice_prescribed_mod due to the intent(out) variable RC not being given an explicit value. This is an operational requirement for NOAA. Fix bug arising from missing j-loop index in ice_import_export

)

* fix dummy arguments w/o values
* fix missing j-loop in export 4d
Comment thread cicecore/drivers/nuopc/cmeps/ice_import_export.F90
@apcraig apcraig merged commit 29c7bcf into CICE-Consortium:main Apr 15, 2024
anton-seaice pushed a commit to ACCESS-NRI/CICE that referenced this pull request May 1, 2024
…sortium#75) (CICE-Consortium#944)

* fix dummy arguments w/o values
* fix missing j-loop in export 4d

Resolve compiler warning arising from ice_prescribed_mod due to the intent(out) variable RC not being given an explicit value. This is an operational requirement for NOAA. Fix bug arising from missing j-loop index in ice_import_export
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants