View Issue Details

IDProjectCategoryView StatusLast Update
0030234CommunityOCCT:Meshpublic2018-11-11 12:00
Reporteradrien_schvalberg_technodigit Assigned Tobugmaster  
PrioritynormalSeveritymajor 
Status closedResolutionfixed 
PlatformWindowsOSVC++ 2015 
Product Version7.3.0 
Target Version7.4.0Fixed in Version7.4.0 
Summary0030234: BRepMesh_IncrementalMesh fail to discretize some zero-length free edges
Descriptionconditions:
- enable Standard_ConstructionError exceptions in config
- set the BRepMesh_IncrementalMesh to use relative deflection
- try to mesh a shape with at least a zero-length free edge (for instance an edge with a 0-radius circle, as it can be found in the outer wire of a sphere)

Then a Standard_ConstructionError exception will be raised inside BRepMesh_IncrementalMesh::discretizeFreeEdges() > CPnts_TangentialDeflection::CPnts_TangentialDeflection() > GCPnts_TangentialDeflection::Initialize()

In my test, with a empty circle for instance, the computed deflection for this edge is less than Precision::Confusion(), which explains the exception.

one possible fix would be to modify BRepMesh_ShapeTool::RelativeEdgeDeflection to have a minimum value strictly greater than Precison::Confusion()
Steps To Reproducerestore empty_circle.brep a
incmesh a 0.005 -relative
TagsNo tags attached.
Test case numberbugs/mesh/bug30234

Attached Files

  • empty_circle.brep (304 bytes)

Relationships

related to 0030345 closedbugmaster Open CASCADE [REGRESSION] Mesh, BRepMesh_CurveTessellator - GCPnts_TangentialDeflection throws Standard_ConstructionError 

Activities

oan

2018-10-11 14:24

developer   ~0079889

Dear adrien_schvalberg_technodigit,

please attach BREP model with described behaviour.

adrien_schvalberg_technodigit

2018-10-11 15:21

reporter  

empty_circle.brep (304 bytes)

adrien_schvalberg_technodigit

2018-10-11 15:23

reporter   ~0079894

load the brep (a 0-radius circle at origin),
test discretization with
BRepMesh_IncrementalMesh mesher(theEdge, 0.005, Standard_True);

oan

2018-10-11 18:00

developer   ~0079906

Thanks a lot for the use case.
Seems the problem is not reproduced using patch 0026106.
However, it has not been integrated to master yet.

adrien_schvalberg_technodigit

2018-10-11 18:09

reporter   ~0079907

Thanks for the update.

to be sure: as I already reported it, you must have an OCC config with Standard_ConstructionError exceptions enabled (ie with #if !defined No_Exception && !defined No_Standard_ConstructionError directive evaluating to true) to reproduce the problem

oan

2018-10-11 18:24

developer   ~0079908

Great notice - I had different configs for master build and patch.

So, unfortunately, it does not solve the problem(
We will investigate it.

Thanks for the details!

adrien_schvalberg_technodigit

2018-10-16 10:19

reporter   ~0079997

I think I'm mistakenly assigned to this issue. I didn't notice since when...

git

2018-11-08 15:20

administrator   ~0080959

Branch CR30234 has been created by nbv.

SHA-1: 4fffe4945e69a4a879a27401c8e4bcf231ffdae5


Detailed log of new commits:

Author: nbv
Date: Thu Nov 8 15:18:16 2018 +0300

    0030234: BRepMesh_IncrementalMesh fail to discretize some zero-length free edges
    
    Test case for the issue has been created.
    This problem is fixed by the patch 0030345.

nbv

2018-11-08 15:55

developer   ~0080963

Dear Mikhail,

Please review the branch CR30234.

Test results:
Linux: http://jenkins-test-12.nnov.opencascade.com/view/CR30234-master_NBV/view/TESTING/job/CR30234-master_NBV-OCCT-Debian80-64-opt-test-restart/HTML_20Report/

Windows: http://jenkins-test-12.nnov.opencascade.com/view/CR30234-master_NBV/view/TESTING/job/CR30234-master_NBV-OCCT-Windows-64-VC14-opt-test-restart/HTML_20Report/

msv

2018-11-08 18:25

developer   ~0080989

Reviewed.

bugmaster

2018-11-09 15:20

administrator   ~0081006

Adding new test case

git

2018-11-11 12:00

administrator   ~0081023

Branch CR30234 has been deleted by inv.

SHA-1: 4fffe4945e69a4a879a27401c8e4bcf231ffdae5

Related Changesets

occt: master 1f17d5fb

2018-11-08 12:18:16

nbv


Committer: bugmaster Details Diff
0030234: BRepMesh_IncrementalMesh fail to discretize some zero-length free edges

Test case for the issue has been created.
This problem is fixed by the patch 0030345.
Affected Issues
0030234
add - tests/bugs/mesh/bug30234 Diff File

Issue History

Date Modified Username Field Change
2018-10-11 13:39 adrien_schvalberg_technodigit New Issue
2018-10-11 13:39 adrien_schvalberg_technodigit Assigned To => oan
2018-10-11 14:24 oan Note Added: 0079889
2018-10-11 14:29 oan Assigned To oan => adrien_schvalberg_technodigit
2018-10-11 14:29 oan Status new => feedback
2018-10-11 15:21 adrien_schvalberg_technodigit File Added: empty_circle.brep
2018-10-11 15:23 adrien_schvalberg_technodigit Note Added: 0079894
2018-10-11 17:59 oan Relationship added related to 0026106
2018-10-11 18:00 oan Note Added: 0079906
2018-10-11 18:03 oan Steps to Reproduce Updated
2018-10-11 18:09 adrien_schvalberg_technodigit Note Added: 0079907
2018-10-11 18:19 oan Relationship deleted related to 0026106
2018-10-11 18:24 oan Note Added: 0079908
2018-10-16 10:19 adrien_schvalberg_technodigit Note Added: 0079997
2018-10-16 14:14 oan Assigned To adrien_schvalberg_technodigit => oan
2018-10-16 14:14 oan Status feedback => assigned
2018-11-06 17:59 oan Relationship added related to 0030345
2018-11-08 13:59 oan Assigned To oan => nbv
2018-11-08 15:20 git Note Added: 0080959
2018-11-08 15:55 nbv Note Added: 0080963
2018-11-08 15:55 nbv Assigned To nbv => msv
2018-11-08 15:55 nbv Status assigned => resolved
2018-11-08 18:25 msv Note Added: 0080989
2018-11-08 18:25 msv Assigned To msv => bugmaster
2018-11-08 18:25 msv Status resolved => reviewed
2018-11-09 15:02 bugmaster Target Version => 7.4.0
2018-11-09 15:20 bugmaster Test case number => bugs/mesh/bug30234
2018-11-09 15:20 bugmaster Note Added: 0081006
2018-11-09 15:20 bugmaster Status reviewed => tested
2018-11-11 11:52 bugmaster Changeset attached => occt master 1f17d5fb
2018-11-11 11:52 bugmaster Status tested => verified
2018-11-11 11:52 bugmaster Resolution open => fixed
2018-11-11 12:00 git Note Added: 0081023