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

MSW: ICD segment/branches WELSEGS lengths should be in absolute for ABS #6074

Closed
hhgs opened this issue Jun 12, 2020 · 0 comments · Fixed by #6129
Closed

MSW: ICD segment/branches WELSEGS lengths should be in absolute for ABS #6074

hhgs opened this issue Jun 12, 2020 · 0 comments · Fixed by #6129
Assignees
@magnesj
Labels
BugInRelease Bug in an official release
Milestone
Patch release 202...

Comments

@hhgs
Copy link
Member

hhgs commented Jun 12, 2020

When MSW is reported with ABS length, ICD segm/branches needs to be reported with absolute length as well:

WELSEGS
71 71 1 70 4618.66959 3859.01626 0.10100 0.0000100 /
72 72 1 71 4621.39489 3861.64841 0.10100 0.0000100 /
-- Combined Valve for segment #2 with contribution from: 2 ICDs: ...
73 73 2 2 0.10000 0 0.10100 0.0000100 /
-- Combined Valve for segment #3 with contribution from: 2 ICDs: ...
74 74 3 3 0.10000 0 0.10100 0.0000100 /
@hhgs hhgs added the BugInRelease Bug in an official release label Jun 12, 2020
@hhgs hhgs added this to the Maintenance 2020 - Sprint 5 milestone Jun 12, 2020
@magnesj magnesj self-assigned this Jun 22, 2020
magnesj added a commit that referenced this issue Jun 22, 2020
@magnesj
#6074 MSW: Add helper function to compute TVD
72e2e97
magnesj added a commit that referenced this issue Jun 22, 2020
@magnesj
#6074 MSW: Add export of both ABS and INC for valves
b66d53a
@magnesj magnesj mentioned this issue Jun 22, 2020
Merged
magnesj added a commit that referenced this issue Jun 22, 2020
@magnesj
#6074 MSW: Add helper function to compute TVD
a8d25c1
magnesj added a commit that referenced this issue Jun 22, 2020
@magnesj
#6074 MSW: Add export of both ABS and INC for valves
c531c94
magnesj added a commit that referenced this issue Jun 29, 2020
@magnesj
#6074 MSW: Add helper function to compute TVD
99c40df
magnesj added a commit that referenced this issue Jun 29, 2020
@magnesj
#6074 MSW: Add export of both ABS and INC for valves
6ee877a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@magnesj magnesj

Labels
BugInRelease Bug in an official release
Projects
None yet
Milestone
Patch release 2020.04.2
Development

Successfully merging a pull request may close this issue.

Several fixes for MSW and valve export OPM/ResInsight
2 participants
@hhgs @magnesj