Currently, providing face (lower case) in the FAULTS section is not allowed and results in an aborted simulation. This issue was discovered while running a test case that uses the FAULTS and CARFIN keywords.
There is an open PR addressing the related test failure in opm-tests (OPM/opm-tests#1485).
Other commercial simulators are case-insensitive in this context and accept both lower- and upper-case input.
The purpose of opening this issue is to discuss whether OPM should also support lower-case input here, and potentially fix this in opm-common by normalizing (e.g., capitalizing) the string when reading the deck.
Currently, providing face (lower case) in the FAULTS section is not allowed and results in an aborted simulation. This issue was discovered while running a test case that uses the FAULTS and CARFIN keywords.
There is an open PR addressing the related test failure in opm-tests (OPM/opm-tests#1485).
Other commercial simulators are case-insensitive in this context and accept both lower- and upper-case input.
The purpose of opening this issue is to discuss whether OPM should also support lower-case input here, and potentially fix this in opm-common by normalizing (e.g., capitalizing) the string when reading the deck.