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.

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

fix: Emit interpolated curve information #2205

Merged
merged 9 commits into from
Aug 28, 2024

Conversation

Anders2303
Copy link
Collaborator

@Anders2303 Anders2303 commented Aug 27, 2024

Solves #2196

  1. Moved the "fillInfo" computation from the WellLogInfoPanel component up to the WellLogViewer and SyncLogViewer core components
  2. Added a onInfoFilled event to the CallbackManager, and made the info-panel listen to that instead
  3. Made the viewer-components emit a onInfoFilled event, to expose the interpolated data to external components
    1. Story-book example at /story/welllogviewer-demo-welllogviewer--on-info-filled-event

@Anders2303 Anders2303 changed the title fix; Emit interpolated curve information fix: Emit interpolated curve information Aug 27, 2024
@hkfb hkfb requested review from Vladimir-Kokin and mirisb August 27, 2024 11:16
@hkfb hkfb merged commit f996eea into equinor:master Aug 28, 2024
10 checks passed
hkfb pushed a commit that referenced this pull request Aug 28, 2024
@hkfb
Copy link
Collaborator

hkfb commented Aug 28, 2024

🎉 This issue has been resolved in version well-log-viewer@1.12.5 🎉

The release is available on GitHub release

@hkfb hkfb added the released label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants