Skip to content

Leave holes open in the CADSupport display mesh - #16013

Open
sawenzel wants to merge 1 commit into
AliceO2Group:devfrom
sawenzel:swenzel/cadsupport-display-holes
Open

sawenzel wants to merge 1 commit into
AliceO2Group:devfrom
sawenzel:swenzel/cadsupport-display-holes

Conversation

@sawenzel

Copy link
Copy Markdown
Collaborator

This fixes a problem in the display mesh of O2BVHSurfaceSolid and adds a unit test.

  • Faces with inner wires were ear-clipped from their outer wire only, so holes and pipe bores were drawn filled.
  • Each hole is now bridged into the outer ring at a visible vertex before ear-clipping.
  • This applies to planar, curved-planar and wire-trimmed quadric faces.
  • Faces without holes keep their triangles unchanged.
  • Holes that touch or cross a wire cannot be bridged, so such faces are still drawn filled.

This fixes a problem in the display mesh of O2BVHSurfaceSolid and adds a unit test.

- Faces with inner wires were ear-clipped from their outer wire only, so holes and pipe bores were drawn filled.
- Each hole is now bridged into the outer ring at a visible vertex before ear-clipping.
- This applies to planar, curved-planar and wire-trimmed quadric faces.
- Faces without holes keep their triangles unchanged.
- Holes that touch or cross a wire cannot be bridged, so such faces are still drawn filled.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sawenzel
sawenzel requested a review from a team as a code owner October 27, 2026 08:21

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant