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

Linked Views: Real range filter syncronization between Geomech-Eclipse views #540

Closed
JacobStoren opened this issue Sep 23, 2015 · 3 comments
Closed

Linked Views: Real range filter syncronization between Geomech-Eclipse views #540

JacobStoren opened this issue Sep 23, 2015 · 3 comments
Assignees
@JacobStoren
Labels
Enhancement An addition that can be observed by the user
Milestone
Geom Sprint 7

Comments

@JacobStoren
Copy link
Member

We need the real ijk-based mapping to be able to apply a range filter across to a corresponding Eclipse/Geomech grid.
We need this to be able to overwrite the range filter in the dependent case when unlinking.
(if requested)
In this way the user can do interactve ijk filtering, in both cases simultaniously. Then, at will, unlink the views and continue from the same "spot" in both cases.
@JacobStoren JacobStoren added the Enhancement An addition that can be observed by the user label Sep 23, 2015
@JacobStoren JacobStoren added this to the Geom Sprint 7 milestone Sep 23, 2015
@JacobStoren
Copy link
Member Author

  • Establish the plain mapping (Offset and axis rotation)
  • Enable use of that mapping in linked rangefilters as a start
  • Establish improved mapping: Intevals of I layers matching I layers and the same for J and K.
    To handle added and removed layers
  • Use the improved mapping in the linked rangefilters
  • Copy rangefilters with remapping to use when unlinking views.

@JacobStoren
Copy link
Member Author

The algorithm behind this is now rethinked. We will instead of trying to calculate the correct mapping up front, calculate the start/end cell of each applied range filter based on geometrical and closest cell match dynamically.

@JacobStoren JacobStoren self-assigned this Oct 15, 2015
JacobStoren added a commit that referenced this issue Oct 15, 2015
@JacobStoren
(#540) Started to make single-cell best-match
707ddf1
JacobStoren added a commit that referenced this issue Oct 20, 2015
@JacobStoren
(#540) WIP: Implemented use of arbitrary exact match diagonal in rang…
c023bc9 
…e filter mapping.

Now we are ready to start testing.
JacobStoren added a commit that referenced this issue Oct 20, 2015
@JacobStoren
(#540) WIP: Split the rangefilter mapping code into files
a5ef83a 
Preparing for assembling it all.
JacobStoren added a commit that referenced this issue Oct 20, 2015
@JacobStoren
(#540) Added interface for fem to ecl range filter mapping as well.
4fc0553
JacobStoren added a commit that referenced this issue Oct 20, 2015
@JacobStoren
(#540) Override range filter update system improved
8b894d8 
Rename of forgotten viewLink ->viewController
Remember to update group of linked views before adding a new dependent view.
JacobStoren added a commit that referenced this issue Oct 20, 2015
@JacobStoren
(#540) Wip: Range filter mapping alive! Needs more adjustments.
58363db 
Fem to Ecl is not perfect.
JacobStoren added a commit that referenced this issue Oct 21, 2015
@JacobStoren
(#540) Wip: Preparing to Extract range endpoint calculation
53826f4 
We neeed to clamp fem range to the extents of the ecl grid within the
fem grid before trying to map fem to ecl. So we need to refactor a bit.
JacobStoren added a commit that referenced this issue Oct 21, 2015
@JacobStoren
(#540) Range filter mapping now works fem->ecl and opposite.
00b56b7 
We need to test more, but now it is getting there.
JacobStoren added a commit that referenced this issue Oct 22, 2015
@JacobStoren
(#540) Guarding against crash when opening project with range filter …
e094ca8 
…override.
JacobStoren added a commit that referenced this issue Oct 22, 2015
@JacobStoren
(#540) Improved handling range filter endpoints in none match cells
9d106ec
JacobStoren added a commit that referenced this issue Oct 22, 2015
@JacobStoren
(#540) Range filter matching: Prioritize cell matches with collapsed …
44d1aaa 
…eclipse cells less.

To make better matches when one or more corner of the range is in a
collapsed cells area
JacobStoren added a commit that referenced this issue Oct 22, 2015
@JacobStoren
(#540) Removed the Visible Cells Sync from Gui.
b15b23b 
Moved UpateOverrides from init afterRead to OpenProject
@JacobStoren
Copy link
Member Author

Now we have something that works fairly robust.
Mapping from Geomech to Eclipse is not as robust as the other way around.
The main weakness is when several range filter "corner" cells are within an area of collapsed eclipse cells.

This becomes a more severe problem for geom to ecl mapping when the eclipse cells at the reservoir edge is collapsed, because we need the eclipse in goemech box for all the calculations.

It is possible to improve this some more, but we need to see how it performs in the wild first.
Will make separate issues for improvement as it proves neccessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JacobStoren JacobStoren

Labels
Enhancement An addition that can be observed by the user
Projects
None yet
Milestone
Geom Sprint 7
Development

No branches or pull requests
1 participant
@JacobStoren