Skip to content

Conversation

@xylar
Copy link
Collaborator

@xylar xylar commented May 14, 2020

It previously pointed to the old (E3SM v1) location for the QU240 test case with ice-shelf cavities.

Also, fix the name of the LIGHT regression suite.

xylar added 2 commits May 14, 2020 04:19
It previously pointed to the old (E3SM v1) location for the
QU240 test case with ice-shelf cavities
@xylar
Copy link
Collaborator Author

xylar commented May 14, 2020

Testing

I ran the land-ice regression suite successfully to completion on Grizzly.

I verified that the name of the resulting python script is not correct for the LIGHT regression suite but I didn't run it. (One of its test cases is broken anyway, see #560.)

Copy link
Contributor

@mark-petersen mark-petersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved by visual inspection. Thanks for fixing it.

@mark-petersen mark-petersen merged commit cc4b1ef into MPAS-Dev:ocean/develop May 19, 2020
@xylar xylar deleted the fix_land_ice_fluxes_regression branch May 19, 2020 13:57
caozd999 pushed a commit to caozd999/MPAS-Model that referenced this pull request Jan 14, 2021
…an/develop

Fix land-ice fluxes regression suite MPAS-Dev#561

It previously pointed to the old (E3SM v1) location for the QU240 test
case with ice-shelf cavities.

Also, fix the name of the LIGHT regression suite.
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