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

Issues with RFT Plots Visualization #11069

Closed
cutn94 opened this issue Jan 16, 2024 · 3 comments · Fixed by #11201
Closed

Issues with RFT Plots Visualization #11069

cutn94 opened this issue Jan 16, 2024 · 3 comments · Fixed by #11201
Assignees
@magnesj
Labels
Bug Bug not yet in an official release
Milestone
Maintenance 2024 ...

Comments

@cutn94
Copy link

cutn94 commented Jan 16, 2024

Hei, it seems that there are a few minor issues regarding the RFT plot visualization. The issues are listed below:

  1. As attached, in RFT plot, the pressure curve is not continuous. However, as I check the plot data, there are values being shown. Is there any possibility that I might have missed something when doing the plot?
  2. As you compared the well diagram with RFT, the topology is not correctly represented.

RFT
well diagram

Please do let me know if you need further elaboration of the issues. Thank you.
@magnesj
Copy link
Member

magnesj commented Jan 18, 2024

Thanks for your report. To further investigate this issue, we will need a test case to reproduce this behaviour. Please upload the required files, or send directly to me.

@hrsut
Copy link

hrsut commented Jan 18, 2024

Hi Magne, I will prepare some test case and the true examples, will reach you out later :)

@magnesj magnesj added the Bug Bug not yet in an official release label Jan 23, 2024
@magnesj magnesj added this to the Maintenance 2024 - Sprint 01 milestone Jan 23, 2024
@magnesj magnesj self-assigned this Feb 13, 2024
magnesj added a commit that referenced this issue Feb 14, 2024
@magnesj
#11069 RFT segment topology
f3cf772 
The segment symbol is displayed once per segment. Previous code used segment start as symbol location. Add support for selecting the location for the segment indicator, either start, mid or end.

When reading segment property values, always use segment end as the curve is plotted using step left.
@magnesj magnesj mentioned this issue Feb 14, 2024
Merged
@magnesj
Copy link
Member

magnesj commented Feb 14, 2024

When ResInsight draws a curve, invalid values are removed. This is the reason why the symbols appear in the plot. During this process the MD of the invalid point is also removed, so it is not possible to plot the full segment length.

magnesj added a commit that referenced this issue Feb 15, 2024
@magnesj
#11069 RFT segment topology
44617ae 
The segment symbol is displayed once per segment. Previous code used segment start as symbol location. Add support for selecting the location for the segment indicator, either start, mid or end.

When reading segment property values, always use segment end as the curve is plotted using step left.
@github-actions github-actions bot added the PendingRelease Issues is fixed and will be available in next release label Feb 15, 2024
@magnesj magnesj removed the PendingRelease Issues is fixed and will be available in next release label Mar 25, 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 02
Development

Successfully merging a pull request may close this issue.

RFT segment topology OPM/ResInsight
3 participants
@magnesj @hrsut @cutn94