search

MPAS-Dev / MPAS-Tools

MPAS Tools Repository
Other
37 stars 63 forks source link

Replace `$` with `$$` when writing config options #474

Closed xylar closed 2 years ago

xylar commented 2 years ago

This is necessary because we always use extended interpolation when we read config files, and a single $ is interpreted as a reference to another config option.

closes #473

xylar commented 2 years ago

Testing

I tested MPAS-Analysis with this fix. Without the fix, the output config files contain values with $ and with the fix they instead have $$ as expected. Further, I made sure that I could run mpas_analysis with the config file written out by a previous run (altering the output path to avoid collisions). Both tests completed successfully.

I ran the ocean pr test suite from compass using this version of MPAS-Tools and everything ran fine. This is not surprising since there are are no $$ in the config files.