View Issue Details

Related ChangesetsJump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0023202CommunityOCCT:Data Exchangepublic2012-06-14 14:282016-04-20 15:49
Reportergoeranbarz Assigned Toabv 
PrioritynormalSeverityminor 
Status closedResolutionfixed 
PlatformWindowsOSVC++ 2010 
Product Version6.5.3 
Target Version7.0.0Fixed in Version7.0.0 
Summary0023202: BRepMesh of face fails
DescriptionWhen I try to Mesh the attached step file, the Meshing fails for face 714
Steps To ReproduceHandle(TDocStd_Document) hDoc;
  XCAFApp_Application::GetApplication()->NewDocument("MDTV_XCAF", hDoc);
  STEPCAFControl_Reader reader;
  reader.SetColorMode(true);
  reader.ReadFile(strFileName);
  reader.Transfer(hDoc);
  
 for (TDF_ChildIterator it(XCAFDoc_DocumentTool::ShapesLabel(hDoc->Main())); it.More() && !progress.canceled(); it.Next())
  {
  if (XCAFDoc_ShapeTool::IsSimpleShape(it.Value()))
  {
  TopoDS_Shape shapeCurrent;
  XCAFDoc_ShapeTool::GetShape(it.Value(), shapeCurrent);
  
 TopExp_Explorer expFace;
  expFace.Init(shapeCurrent, TopAbs_FACE);
  int nMin(714), nMax(714); //The exception occurs in face 714
  for (Standard_Integer numFace = 1; expFace.More(); expFace.Next(), ++numFace)
  {
  if (numFace < nMin || numFace > nMax)
  continue;
  TopoDS_Face myFace = TopoDS::Face(expFace.Current());
  Handle(Poly_Triangulation) myT = BRep_Tool::Triangulation(myFace, myLocation);
  if (myT.IsNull())
  {
  BRepMesh_IncrementalMesh Mesh(myFace, 0.7, Standard_False, 0.75);
  myT = BRep_Tool::Triangulation(myFace, myLocation);//Triangulation is Null
  }
  }
  }
  }
 
TagsNo tags attached.
Test case numberbugs mesh bug23202

Attached Files

  • Leuchte.stp (1,755,216 bytes)
  • axo.png (5,076 bytes)
  • v2d.png (1,455 bytes)

Relationships

related to 0024595 closedbugmaster STEP import missing surfaces 

Activities

goeranbarz

2012-06-14 14:28

reporter  

Leuchte.stp (1,755,216 bytes)

oan

2013-03-04 17:40

developer   ~0023579

Checkshape and incmesh commands indicate the open wire problem on face #714. It is obvious in 2d (see v2d.png) - the whole face is represented by a single unclosed curve. The following script can be used to reproduce these results:

pload ALL

stepread Leuchte.stp a *
explode a_1 f
renamevar a_1_714 b

donly b
small
fit
isos b 10

incmesh b 0.1

# Incremental Mesh, multi-threading OFF
# Meshing statuses: OpenWire

checkshape b

# On Shape faulty_1 :
# BRepCheck_UnorientableShape
# Shape faulty_2 on shape faulty_1 :
# BRepCheck_NotClosed
#
# Faulty shapes in variables faulty_1 to faulty_2

v2d2
pcurve b
2dfit

I suppose it is out of responsibility of the BRepMesh and some troubles during importation process of the STEP file are the cause of this problem. I propose to move it to the DataExchange category.

Dear ABV,
could you express your suggestions?

oan

2013-03-04 17:41

developer  

axo.png (5,076 bytes)

oan

2013-03-04 17:41

developer  

v2d.png (1,455 bytes)

abv

2013-03-05 10:33

manager   ~0023582

I believe the face #27244 in the STEP file is incorrectly defined: its bounding loop contains only VERTEX_LOOP which is (as far as I remember) is allowed to be used only on complete spheres. The actual surface of that face is B-spline representing half-sphere (with a pole inside).

I suggest this file should be checked for reading by other (third-party) applications, and only if other systems can read this face correctly, we shall consider improving OCCT to handle such cases.

goeranbarz

2013-03-07 19:15

reporter   ~0023634

The halfsphere is imported correctly by "Autodesk Inventor 2012". The older version "Autodesk Inventor 2008" wasn't able to import the half-sphere.

oan

2016-02-15 15:29

developer   ~0050757

Dear Bugmaster,

it seems that this issue has been fixed by 0024595. Please create test case and verify.

git

2016-02-17 13:25

administrator   ~0050868

Branch CR23202 has been created by apn.

SHA-1: 59bb2e1ead78bcaa4029f60c84536ec208f5d75f


Detailed log of new commits:

Author: apn
Date: Wed Feb 17 13:24:44 2016 +0300

    0023202: BRepMesh of face fails
    
    Added test case bugs/mesh/bug23202

apn

2016-02-17 13:26

administrator   ~0050869

Test case bugs mesh bug23202 was added to CR23202
http://occt-tests/bug23202_Lin/bugs/mesh/bug23202.html
http://occt-tests/bug23202_WNT/bugs/mesh/bug23202.html

Dear oan, please review test case.

oan

2016-02-17 13:47

developer   ~0050870

Dear Andrey,

could you please add an additional check like checktriarea.

git

2016-02-17 14:25

administrator   ~0050872

Branch CR23202 has been updated forcibly by apn.

SHA-1: 251d2ead21a3037d3ae18881e93d85fa6d35bf96

apn

2016-02-17 14:26

administrator   ~0050873

Done.

git

2016-04-17 13:18

administrator   ~0052828

Branch CR23202 has been deleted by kgv.

SHA-1: 251d2ead21a3037d3ae18881e93d85fa6d35bf96

Related Changesets

occt: master 85b47ba4

2016-02-17 10:24:44

abv


Committer: abv Details Diff		 0023202: BRepMesh of face fails

Added test case bugs/mesh/bug23202		 Affected Issues
0023202
add - tests/bugs/mesh/bug23202 Diff File

Issue History

Date Modified Username Field Change
2012-06-14 14:28 goeranbarz New Issue
2012-06-14 14:28 goeranbarz Assigned To => jgv
2012-06-14 14:28 goeranbarz File Added: Leuchte.stp
2012-11-09 09:54 abv Category OCCT:Modeling Algorithms => OCCT:Mesh
2013-03-04 17:40 oan Note Added: 0023579
2013-03-04 17:41 oan File Added: axo.png
2013-03-04 17:41 oan File Added: v2d.png
2013-03-04 17:41 oan Assigned To jgv => abv
2013-03-04 17:41 oan Status new => feedback
2013-03-05 10:33 abv Note Added: 0023582
2013-03-05 10:33 abv Assigned To abv => goeranbarz
2013-03-05 10:33 abv Target Version => Unscheduled
2013-03-07 19:15 goeranbarz Note Added: 0023634
2013-03-07 19:15 goeranbarz Status feedback => assigned
2013-03-29 11:39 oan Category OCCT:Mesh => OCCT:Data Exchange
2016-02-15 15:17 abv Relationship added related to 0024595
2016-02-15 15:28 oan Target Version Unscheduled => 7.0.0
2016-02-15 15:29 oan Note Added: 0050757
2016-02-15 15:29 oan Assigned To goeranbarz => apn
2016-02-15 15:29 oan Status assigned => feedback
2016-02-17 13:25 git Note Added: 0050868
2016-02-17 13:26 apn Note Added: 0050869
2016-02-17 13:26 apn Test case number => bugs mesh bug23202
2016-02-17 13:26 apn Assigned To apn => oan
2016-02-17 13:47 oan Note Added: 0050870
2016-02-17 13:47 oan Assigned To oan => apn
2016-02-17 13:47 oan Status feedback => assigned
2016-02-17 14:25 git Note Added: 0050872
2016-02-17 14:26 apn Note Added: 0050873
2016-02-17 14:27 apn Assigned To apn => bugmaster
2016-02-17 14:27 apn Status assigned => resolved
2016-02-17 14:27 apn Status resolved => reviewed
2016-02-17 14:27 apn Status reviewed => tested
2016-02-21 08:52 abv Changeset attached => occt master 85b47ba4
2016-02-21 08:52 abv Assigned To bugmaster => abv
2016-02-21 08:52 abv Status tested => verified
2016-02-21 08:52 abv Resolution open => fixed
2016-04-17 13:18 git Note Added: 0052828
2016-04-20 15:43 aiv Fixed in Version => 7.0.0
2016-04-20 15:49 aiv Status verified => closed