Skip to content

880 Data Provider Framework in Intersection Module #948

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

Open
wants to merge 53 commits into
base: main
Choose a base branch
from

Conversation

jorgenherje
Copy link
Collaborator

@jorgenherje jorgenherje commented Apr 30, 2025

Apply data provider framework to Intersection Module

This is a first version only supporting 1 view using the EsvIntersection-component.

Made settings and providers for new functionality. Adjustments to existing SeismicLayer to perform async generating seismic image.

Fixes made compared to old version:

  • Corrected incorrect resampling of polyline: Now it resamples per polyline section, and ensures last point being added. Thereby there is no cutoff
  • Ensures extensionLength is only used for wellbores.

Adjustment/fixes in back-end:

  • Adjust/move endpoints in back-end. Move end-points and utils in form surface router to well router for correct placement
  • Remove unused/unnecessary endpoint
  • Fix error message in surface endpoint (print error text)

Closes: #880

@jorgenherje jorgenherje added the enhancement New feature or request label Apr 30, 2025
@jorgenherje jorgenherje self-assigned this Apr 30, 2025
@jorgenherje jorgenherje force-pushed the 880-intersection-module-layer-framework-NEW branch 4 times, most recently from 6aef083 to b58963c Compare May 8, 2025 14:02
- Fixes after rebase
- WIP
- Make correct viewport
- Fix correct viewport for `handleFitInViewClick`, make use of bounds instead of referenceSystem, as bounds include extensionLength
Change: Remove conditional rendering for ViewportWrapper
Note: Does not explain why context issue occurs
- Sort available settings arrays for options ordered alphabetically
- Make use of TimeOrIntervalSetting for formatting of label in Dropdown
- Handle inf, nan, MAX_VALUE etc to ensure valid bounds and viewport
@jorgenherje jorgenherje force-pushed the 880-intersection-module-layer-framework-NEW branch from b58963c to b4fd8c9 Compare May 12, 2025 12:12
- Undo incorrect package update
- Update autogen code
- Add adjusted preview code done in original module code
@jorgenherje jorgenherje force-pushed the 880-intersection-module-layer-framework-NEW branch from b4fd8c9 to dcdc748 Compare May 12, 2025 12:19
@jorgenherje jorgenherje marked this pull request as ready for review May 16, 2025 10:53
onRescale did not ensure that image was generated. Update to ensure preRender logic is run and image is generated async when needed
Adjust naming for consistency
Copy link
Collaborator

@Anders2303 Anders2303 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work. Found some various smaller issues, but overall everything looks good.

I'll take a look at the running instance tomorrow to check the UX

@Anders2303
Copy link
Collaborator

Found duplicate well-picks on No 34/4-D-3 AH
image

@jorgenherje
Copy link
Collaborator Author

Found duplicate well-picks on No 34/4-D-3 AH image

Updated the DrilledWellborePicksSetting to have unique pickIdentifiers in the options. Then no duplicates exist among options. Thereafter we find all elements with matching the selected pickIdentifiers among the availableValues in handleChange function, to ensure all picks are selected and provided to the user.

Refactor code in DataProvidersWrapper:
- Refactor into utility functions
- Add states for reference stability
- Create new viewport when changing intersection or extension length
- Handle vertical scale/zFactor when creating viewport.
- Move calculation of viewport into wrapper
- Refocus view when changing intersection/extension
- Introduce fit in view mode
- Do not reset/refocus view when adding layers etc unless fit in view mode is on
@jorgenherje jorgenherje requested a review from Anders2303 May 27, 2025 08:38
To prevent duplicate div refs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Apply new layer framework to Intersection Module
4 participants