Skip to content

fix(VolumeRendering): on switching dataset, ensure CTF preset valid#656

Merged
floryst merged 1 commit intoKitware:mainfrom
PaulHax:preset-fix
Oct 2, 2024
Merged

fix(VolumeRendering): on switching dataset, ensure CTF preset valid#656
floryst merged 1 commit intoKitware:mainfrom
PaulHax:preset-fix

Conversation

@PaulHax
Copy link
Collaborator

@PaulHax PaulHax commented Oct 2, 2024

Bug reproduction steps:

  1. Load prostate sample
  2. Switch to Rendering tab
  3. Load Fetus sample

Error: VolumeRendering component would get an empty string preset, then fail to get a color transfer function from that.

This change initializes the volume-coloring store for a dataset in all components that depend on volume-coloring.

Bug reproduction steps:
1. Load prostate sample
2. Switch to Rendering tab
3. Load Fetus sample

Error: VolumeRendering component would get an empty string preset, then
fail to get a color transfer function from that.

This change initializes the volume-coloring store for a dataset
in all components that depend on volume-coloring.
@netlify
Copy link

netlify bot commented Oct 2, 2024

Deploy Preview for volview-dev ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 62bf6bd
🔍 Latest deploy log https://app.netlify.com/sites/volview-dev/deploys/66fd9d358cd21400077c57a4
😎 Deploy Preview https://deploy-preview-656--volview-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@floryst floryst added this pull request to the merge queue Oct 2, 2024
Merged via the queue into Kitware:main with commit a7535a4 Oct 2, 2024
PaulHax pushed a commit to PaulHax/VolView that referenced this pull request Apr 24, 2025
fix(VolumeRendering): on switching dataset, ensure CTF preset valid
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants