Switch `MappingFileStep` to be a command-line step

E3SM-Project / polaris

Testing and analysis for OMEGA, MPAS-Ocean, MALI and MPAS-Seaice

BSD 3-Clause "New" or "Revised" License

6 stars 13 forks source link

Closed xylar closed 1 year ago

xylar commented 1 year ago

Rather than using pyremap to do a subprocess call, we construct a list of command-line arguments and let polaris take care of the MPI arguments.

Checklist

[x] Developer's Guide has been updated
[x] API documentation in the Developer's Guide (api.md) has any new or modified class, method and/or functions listed
[x] Documentation has been built locally and changes look as expected
[x] Testing comment in the PR documents testing used to verify the changes

xylar commented 1 year ago

I ran the ocean/global_convergence/icos/cosine_bell test case with this branch and it completed successfully (including the *_map steps).

xylar commented 1 year ago

@altheaden, please review as soon as you're allowed.