0030292: Modeling Algorithms - BRepBndLib should avoid using Poly_Polygon3D when called with useTriangulation set to false - MantisBT

MantisBT

Mantis Bug Tracker Workflow

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

ID

Project

Category

View Status

Date Submitted

Last Update

0030292

Open CASCADE

[OCCT] OCCT:Modeling Algorithms

public

2018-10-25 01:18

2018-10-25 11:23

Reporter

kgv

Assigned To

ifv

Priority

normal

Severity

minor

Status

assigned

Resolution

open

Platform

OS

OS Version

Product Version

[OCCT] 7.3.0

Target Version

[OCCT] 7.4.0*

Fixed in Version

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

Relationships

Notes
(0080295) git (administrator) 2018-10-25 10:19	Branch CR30292 has been created by kgv. SHA-1: 5a75e65eb4493ebd520e98507981290da4bcfcc1 Detailed log of new commits: Author: kgv 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

(0080296) msv (developer) 2018-10-25 11:22	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.

(0080297) msv (developer) 2018-10-25 11:23	The non-regression test cases are also needed.

Issue History
Date Modified	Username	Field	Change
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

Copyright © 2000 - 2019 MantisBT Team