Skip to content

fix(ImageResliceMapper): normalize slab sampling direction#3492

Merged
finetjul merged 1 commit intoKitware:masterfrom
daker:fix-imagereslicemapper
Apr 30, 2026
Merged

fix(ImageResliceMapper): normalize slab sampling direction#3492
finetjul merged 1 commit intoKitware:masterfrom
daker:fix-imagereslicemapper

Conversation

@daker
Copy link
Copy Markdown
Collaborator

@daker daker commented Apr 29, 2026

Context

Ensure the slab sampling direction is normalized before computing step sizes and traversal distances.
Previously, normalWCVSOutput was used directly, which could be a non-unit vector. This caused incorrect scaling of sampling steps, leading to inconsistent sampling density and potential visual artifacts when traversing the slab.

Results

Changes

  • Documentation and TypeScript definitions were updated to match those changes

PR and Code Checklist

  • semantic-release commit messages
  • Run npm run reformat to have correctly formatted code

Testing

  • This change adds or fixes unit tests
  • Tested environment:
    • vtk.js:
    • OS:
    • Browser:

Ensure the slab sampling direction is normalized before computing
step sizes and traversal distances.
Previously, normalWCVSOutput was used directly, which could be a
non-unit vector. This caused incorrect scaling of sampling steps,
leading to inconsistent sampling density and potential visual artifacts
when traversing the slab.
Copy link
Copy Markdown
Member

@finetjul finetjul left a comment

Choose a reason for hiding this comment

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

LGTM

@finetjul finetjul added this pull request to the merge queue Apr 30, 2026
Merged via the queue into Kitware:master with commit dfbee4c Apr 30, 2026
2 checks passed
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 35.11.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions Bot added the released Automated label label Apr 30, 2026
@daker daker deleted the fix-imagereslicemapper branch April 30, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Automated label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants