View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0030292||Open CASCADE||OCCT:Modeling Algorithms||public||2018-10-25 01:18||2022-10-24 10:43|
|Summary||0030292: Modeling Algorithms - BRepBndLib should avoid using Poly_Polygon3D when called with useTriangulation set to false|
|Description||BRepBndLib::Add() has flag to either use triangulation when possible or to use geometric data.|
It looks natural that useTriangulation flag affects using any tessellated data for computing bounding box, including Face triangulation as well as Edge polyline, but apparently the latter is not the case.
It is proposed to update implementation and description of the tool to ignore any tessellated data including polygon3d within Edge.
|Tags||No tags attached.|
|Test case number|
Branch CR30292 has been created by kgv.
Detailed log of new commits:
Date: Thu Oct 25 10:13:44 2018 +0300
0030292: Modeling Algorithms - BRepBndLib should avoid using Poly_Polygon3D when called with useTriangulation set to false
It is needed to implement the same logic for edges without faces as for faces: polygon is used if there are no geometry or the flag useTriangulation is true. Geometry is used if the flag useTriangulation is false or there are no polygon.
It seems the proposed patch fixes this well for the method Add(). It is needed to do the same for the method AddOptimal().
Also, it is needed to update the description of the method Add() in the header file.
||The non-regression test cases are also needed.|
|2018-10-25 01:18||kgv||New Issue|
|2018-10-25 01:18||kgv||Assigned To||=> msv|
|2018-10-25 10:19||git||Note Added: 0080295|
|2018-10-25 11:22||msv||Note Added: 0080296|
|2018-10-25 11:22||msv||Assigned To||msv => ifv|
|2018-10-25 11:22||msv||Status||new => assigned|
|2018-10-25 11:23||msv||Note Added: 0080297|
|2019-08-12 18:01||msv||Target Version||7.4.0 => 7.5.0|
|2020-09-14 22:53||msv||Target Version||7.5.0 => 7.6.0|
|2021-08-29 18:51||msv||Target Version||7.6.0 => 7.7.0|
|2022-10-24 10:43||szy||Target Version||7.7.0 => 7.8.0|