Skip to content
This repository was archived by the owner on Oct 23, 2020. It is now read-only.

Conversation

@xylar
Copy link
Contributor

@xylar xylar commented Nov 23, 2016

All calls to MpasMaskCreator.x, MpasMeshConverter.x and MpasCellCuller.x require -i and -o flags to be compatible with changes made to the syntax of these tools in MPAS-Dev/MPAS-Tools#149. Also based on the same PR, the flag for inverse masking in the cell culler has been changed from -i to -v.

This merge also fixes a minor issue with the syntax in calls to the script init_step1.py in some global_ocean test cases.

All calls to MpasMaskCreator.x, MpasMeshConverter.x and
MpasCellCuller.x require -i and -o flags to be compatible with
changes made to the syntax of these tools in the following
pull requrest:
MPAS-Dev/MPAS-Tools#149

Also, fixed a minor issue with the syntax in calls to the script
init_step1.py in some global_ocean test cases
@xylar
Copy link
Contributor Author

xylar commented Nov 23, 2016

Testing

I made sure the nightly regression tests are bit-for-bit identical to ocean/develop with these changes (and using the tools from MPAS-Dev/MPAS-Tools#149).

I don't plan on testing all 56 test cases because the changes in other test cases are exactly identical to those in tests covered by the nightly regression suite.

@xylar
Copy link
Contributor Author

xylar commented Dec 4, 2016

It looks like we decided not to merge MPAS-Dev/MPAS-Tools#149 so I will close this PR, too.

@xylar xylar closed this Dec 4, 2016
@xylar xylar deleted the ocean/change_mesh_conversion_flags branch March 24, 2017 16:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants