Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Noisy warning #3223

Open
alfbr opened this issue Nov 16, 2022 · 2 comments
Open

Noisy warning #3223

alfbr opened this issue Nov 16, 2022 · 2 comments

Comments

@alfbr
Copy link
Member

alfbr commented Nov 16, 2022 •
edited by bska
Loading

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
Copy link
Member

bska commented Nov 16, 2022

Looking through the file history, this message was added in commit 172725a (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
Copy link
Member Author

alfbr commented Nov 16, 2022

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alfbr @bska