-
Notifications
You must be signed in to change notification settings - Fork 107
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.
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
improve ecl_config script error message #5129
Comments
Can anyone help me with this issue so that I can raise a PR to close it. |
what kind of questions do you have, @Jehan-h20220012 ? |
See also: #4241 (Resolving that issue would remove this codepath) |
i haven't gotten into the details of these things yet - @sondreso , does the issue you link to imply that we want to get rid of the script that this issue wants to improve? : ) |
There are essentially two ways of starting the Eclipse simulator in the code today, one code path that interacts with a binary called eclrun (for all versions newer than 2018.2) and another code path that interacts with the binary of the individual releases (versions prior to 2018.2). The error message above comes from the latter. We have confirmation that we don't need to support these older versions in the newest version of ERT, so we can remove this code path altogether. |
nice! closing in favor of #4241 |
Describe the bug
When running some tests (everest), a file is reported as non-executable, which is technically correct, but in fact the file is non-existing. the error message should be improved.
Error message:
Suggested fix: check for missing file in
src/ert/shared/share/ert/forward-models/res/script/ecl_config.py
around line 45.The text was updated successfully, but these errors were encountered: