View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0033632 | Community | OCCT:Modeling Algorithms | public | 2024-03-05 14:21 | 2024-11-16 13:38 |
Reporter | dpasukhi | Assigned To | |||
Priority | normal | Severity | minor | ||
Status | new | Resolution | open | ||
Target Version | Unscheduled | ||||
Summary | 0033632: Modeling Algorithms - BRepTools::OuterWire returns internal wire | ||||
Description | Hello, I have a face including four holes from which I need the outer wire but BRepTools::OuterWire returns the wire of one of the holes as outer wire. I traced the problem to BndLib_Box2dCurve::PerformBSpline where is just blatantly adds poles to the bounding box. This reminded of a post that Roman had done years ago. Romans work around doesn't seem to help you as the presence of triangulation doesn't seem to affect the 2d bounding box calculation. I would assume Roman is correct in speculating that this kind of 'just use the poles, hack' is for performance and any changes to a 'minimum bounding box' would slow things down. Looking at BRepTools::OuterWire: It assigns the first wire and then iterates over the rest of the wires requiring larger boundaries in all directions. So in your case, the first wire is one of the circles that has an inflated v range. The rectangular wires v boundaries are inside of that first circular wire, causing the algo to fail. I think this algo could at least have a smarter first guess using the diagonal of the 2d bounding boxes, without much performance impact, but there would still be cases that will fail as long as the spline bounds are 'inflated'. | ||||
Additional information and documentation updates | https://dev.opencascade.org/content/breptoolsouterwire-returns-wrong-wire#comment-25614 | ||||
Tags | No tags attached. | ||||
Test case number | |||||
Date Modified | Username | Field | Change |
---|---|---|---|
2024-03-05 14:21 | dpasukhi | New Issue | |
2024-03-05 14:21 | dpasukhi | Assigned To | => oan |
2024-03-05 14:21 | dpasukhi | Note Added: 0115231 | |
2024-03-05 14:21 | dpasukhi | File Added: face.brep | |
2024-03-05 14:22 | dpasukhi | Additional Information Updated | |
2024-05-16 13:27 | oan | Assigned To | oan => |