Skip to content

Conversation

@Joeycho
Copy link
Contributor

@Joeycho Joeycho commented Nov 17, 2025

Current:
direction missing.
#3355

PR:
Input image has indexToWorld function. Reuse it.

@Joeycho
Copy link
Contributor Author

Joeycho commented Nov 17, 2025

@finetjul

Copy link
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.

Please reformat your commit message to match semantic release.

@Joeycho Joeycho changed the title use indexToWorld function, rather than separate calculation Fix (ImageMarchingSquares): use indexToWorld function, rather than separate calculation Nov 18, 2025
@Joeycho Joeycho changed the title Fix (ImageMarchingSquares): use indexToWorld function, rather than separate calculation fix (ImageMarchingSquares): use indexToWorld function, rather than separate calculation Nov 18, 2025
@Joeycho Joeycho changed the title fix (ImageMarchingSquares): use indexToWorld function, rather than separate calculation fix(ImageMarchingSquares): use indexToWorld function, rather than separate calculation Nov 18, 2025
@Joeycho Joeycho requested a review from finetjul November 26, 2025 14:06
@Joeycho Joeycho requested a review from finetjul November 28, 2025 14:54
@Joeycho Joeycho requested a review from finetjul December 3, 2025 09:38
Copy link
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

@Joeycho
Copy link
Contributor Author

Joeycho commented Dec 9, 2025

@finetjul time to merge? how can I merge? Or can you do that?

@finetjul finetjul added this pull request to the merge queue Dec 9, 2025
Merged via the queue into Kitware:master with commit 327921d Dec 9, 2025
2 checks passed
@github-actions
Copy link

github-actions bot commented Dec 9, 2025

🎉 This PR is included in version 34.15.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot added the released Automated label label Dec 9, 2025
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