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

BUG: Fixes issue where any component shape was allowed for ComputeTriangleGeometrySizes #1134

Merged

Conversation

imikejackson
Copy link
Contributor

  • Fix issue where any component shape was allowed for the Face Labels input array.
    - Rename filter to ComputeTriangleGeomVolumes

@imikejackson imikejackson self-assigned this Nov 20, 2024
@imikejackson imikejackson added bug Something isn't working documentation Improvements or additions to documentation labels Nov 20, 2024
@imikejackson imikejackson marked this pull request as draft November 23, 2024 15:14
@imikejackson imikejackson force-pushed the topic/triangle_vol_fixes branch 2 times, most recently from a7565c0 to 48cdadc Compare November 23, 2024 22:47
…iangleGeometrySizes.cpp

Signed-off-by: Michael Jackson <[email protected]>
Signed-off-by: Michael Jackson <[email protected]>
(cherry picked from commit ca7b027)
Signed-off-by: Michael Jackson <[email protected]>
(cherry picked from commit 8c0e521)
…array.

	A negative value for the face label when used as an index will cast into an unsigned value
    and bad things are going to happen.

Signed-off-by: Michael Jackson <[email protected]>
Signed-off-by: Michael Jackson <[email protected]>
@imikejackson imikejackson marked this pull request as ready for review November 24, 2024 15:12
@imikejackson imikejackson merged commit 4f44d73 into BlueQuartzSoftware:develop Nov 24, 2024
7 checks passed
@imikejackson imikejackson deleted the topic/triangle_vol_fixes branch November 24, 2024 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants