search

OPM / opm-common

Common components for OPM, in particular build system (cmake).
http://www.opm-project.org
GNU General Public License v3.0
30 stars 112 forks source link

Noisy warning #3223

Open alfbr opened 1 year ago

alfbr commented 1 year ago

It seems a message intended for developers have been implemented outside the logging framework. This is very unfortunate as it confuses users.

See line 174-185 in src/opm/input/eclipse/EclipseState/Tables/SimpleTable.cpp.

bska commented 1 year ago

Looking through the file history, this message was added in commit 172725aef8ed8ea439c29ea5beacd670ddc82a3e (originally PR OPM/opm-parser#970) and then just never updated to use the logging system. The comment states that this condition should be an error, but if it happens in a field case then there's something we're missing.

alfbr commented 1 year ago

It happens on a case here, so we should probably look into exactly what is going on.