COSIMA / om3-utils

Mozilla Public License 2.0
0 stars 2 forks source link

MOM_input parameter conflict written by mom6_input.py #14

Open minghangli-uni opened 1 month ago

minghangli-uni commented 1 month ago

Specifically for

CVMix_CONVECTION%
%CVMix_CONVECTION

mom6_input.py incorrectly parses them as

CVMIX_CONVECTION%
%CVMix_CONVECTION

This pops up a conflict for a model run.

A simple fix is to remove this line for now. mom6_input_str = re.sub(r"((?<=^)|(?<=\n))(\w+%?)", lambda pat: pat.group(2).upper(), mom6_input_str)

micaeljtoliveira commented 1 month ago

There's indeed an issue with the parser, as it uses f90nml, which will treat the input as case insensitive. I need to rewrite this, as there's no "easy" fix. It's been on my TODO list for a while, but had not time to work on it.