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

Fix import of time since simulation start #11092

Closed
magnesj opened this issue Jan 22, 2024 · 0 comments · Fixed by #11093
Closed

Fix import of time since simulation start #11092

magnesj opened this issue Jan 22, 2024 · 0 comments · Fixed by #11093
Assignees
@magnesj
Labels
Bug Bug not yet in an official release
Milestone
Maintenance 2024 ...

Comments

@magnesj
Copy link
Member

magnesj commented Jan 22, 2024 •
edited
Loading

@bska commented on recent changes related to import of time information from restart files. The refactoring read data from the wrong data field as commented here:

018e911#r137546418

See how libecl imports similar data
https://github.com/OPM/ResInsight/blob/dev/ApplicationLibCode/FileInterface/RifEclipseOutputFileTools.cpp#L159-L170

Fix import by reading data from index 0 in DOUBHEAD array.
@magnesj magnesj added the Bug Bug not yet in an official release label Jan 22, 2024
@magnesj magnesj added this to the Maintenance 2024 - Sprint 01 milestone Jan 22, 2024
@magnesj magnesj self-assigned this Jan 22, 2024
magnesj added a commit that referenced this issue Jan 22, 2024
@magnesj
#11092 Fix import of time since simulation start
244e044
@magnesj magnesj mentioned this issue Jan 22, 2024
Merged
magnesj added a commit that referenced this issue Jan 22, 2024
@magnesj
#11092 Fix import of time since simulation start
6260995
@github-actions github-actions bot added the PendingRelease Issues is fixed and will be available in next release label Jan 22, 2024
@magnesj magnesj removed the PendingRelease Issues is fixed and will be available in next release label Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@magnesj magnesj

Labels
Bug Bug not yet in an official release
Projects
None yet
Milestone
Maintenance 2024 - Sprint 01 - 2023.1...
Development

Successfully merging a pull request may close this issue.

Fix import of time since simulation start OPM/ResInsight
1 participant
@magnesj