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.

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

UNRST and EGRID file matching confused by . in basename #2854

Closed
JacobStoren opened this issue May 4, 2018 · 0 comments
Closed

UNRST and EGRID file matching confused by . in basename #2854

JacobStoren opened this issue May 4, 2018 · 0 comments
Assignees
Labels
BugInRelease Bug in an official release

Comments

@JacobStoren
Copy link
Member

If we have "." in the basename of the file set, matching between EGRID and UNRST might get confused.
The results does not match the EGRID case, but rather a different one.

From e-mail frm hhgs fr 04.05.2018 14:46

You can do the following tests:

  1. Copy 2C3_BASE_CP_PLAN6.7.DATA and call it 2C3_BASE_CP_PLAN6.7_TEST.DATA and run again. After it has started (and started properly), you can load the case into Resinsight.
  2. Copy 2C3_BASE_CP_PLAN6.7.DATA and call it TEST DATA. After it has started (and started properly), you can load the case into Resinsight.
  3. Copy 2C3_BASE_CP_PLAN6.7.DATA and call it 2C3_BASE_CP_TEST_PLAN6.7.DATA. After it has started (and started properly), you can load the case into Resinsight.

For 1) you will be able to see RESTART data by 2045, but not for 2) and 3). Seems like something happens when there is a dot in the file name. Perhaps there is something that parses filenames and splits on a sentence?

I've already done steps 1,2 and 3 in the directory I linked you so you can check.

@JacobStoren JacobStoren added the BugInRelease Bug in an official release label May 4, 2018
@bjornjensen bjornjensen self-assigned this May 8, 2018
@lindkvis lindkvis self-assigned this May 14, 2018
lindkvis added a commit that referenced this issue May 14, 2018
* Need to send in QFileInfo::completeBaseName() rather than just baseName()
* The change will cause libECL to match only files where the base name is everything except what is after the LAST  dot.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BugInRelease Bug in an official release
Projects
None yet
Development

No branches or pull requests

4 participants