Salome HOME
Prevent failure of the second compute in case of not closed 2D mesh
[plugins/netgenplugin.git] / src / NETGENPlugin / NETGENPlugin_Remesher_2D.cxx
index dd36c08f4f6b347861cd2f8062a9e1f471046742..03141b112fd2be960cd480ffcf78ca4f27f026ba 100644 (file)
@@ -43,7 +43,7 @@
 #include <occgeom.hpp>
 #include <meshing.hpp>
 #include <stlgeom.hpp>
-//#include <stltool.hxx>
+//#include <stltool.hpp>
 
 #include <boost/container/flat_set.hpp>
 
@@ -56,6 +56,10 @@ using namespace nglib;
 // #endif
 //   extern STLParameters stlparam;
 // }
+namespace nglib
+{
+  extern netgen::Array<netgen::Point<3> > readedges;
+}
 
 namespace
 {
@@ -70,12 +74,14 @@ namespace
     HoleFiller( SMESH_Mesh& meshDS );
     ~HoleFiller();
     void AddHoleBorders( Ng_STL_Geometry * ngStlGeo );
-    void KeepHole() { myHole.clear(); }
+    void KeepHole() { myHole.clear(); myCapElems.clear(); }
+    void ClearCapElements() { myCapElems.clear(); }
 
   private:
-    SMESHDS_Mesh*                        myMeshDS;
-    std::vector< std::vector< gp_XYZ > > myHole;      // initial border nodes
-    std::vector< gp_XYZ >                myInHolePos; // position inside each hole
+    SMESHDS_Mesh*                          myMeshDS;
+    std::vector< std::vector< gp_XYZ > >   myHole;      // initial border nodes
+    std::vector< gp_XYZ >                  myInHolePos; // position inside each hole
+    std::vector< const SMDS_MeshElement* > myCapElems;  // elements closing holes
   };
 
   //================================================================================
@@ -106,9 +112,9 @@ namespace
     if ( !isManifold )
     {
       // set bad faces into a compute error
-      SMESH_ComputeErrorPtr error =
-        SMESH_ComputeError::New( COMPERR_BAD_INPUT_MESH,
-                                 "Non-manifold mesh. Only manifold mesh can be re-meshed");
+      const char* text = "Non-manifold mesh. Only manifold mesh can be re-meshed";
+      SMESH_BadInputElements* error =
+        new SMESH_BadInputElements( myMeshDS, COMPERR_BAD_INPUT_MESH, text );
       SMESH::Controls::MultiConnection2D fun;
       fun.SetMesh( myMeshDS );
       SMDS_ElemIteratorPtr fIt = myMeshDS->elementsIterator( SMDSAbs_Face );
@@ -118,9 +124,49 @@ namespace
         if ( fun.GetValue( f->GetID() ) > 2 )
           error->myBadElements.push_back( f );
       }
-      theMesh.GetSubMesh( theMesh.GetShapeToMesh() )->GetComputeError() = error;
+      theMesh.GetSubMesh( theMesh.GetShapeToMesh() )->GetComputeError().reset( error );
+      throw SALOME_Exception( text );
+    }
+
+    // don't want to sew coincident borders
+    if ( !holes.empty() )
+    {
+      // define tolerance
+      double tol, len, sumLen = 0, minLen = 1e100;
+      int     nbSeg = 0;
+      for ( size_t i = 0; i < holes.size(); ++i )
+      {
+        nbSeg += holes[i].size();
+        SMESH_NodeXYZ p1 = holes[i][0];
+        for ( size_t iP = 1; iP < holes[i].size(); ++iP )
+        {
+          SMESH_NodeXYZ p2 = holes[i][iP];
+          len = ( p1 - p2 ).Modulus();
+          sumLen += len;
+          minLen = Min( minLen, len );
+          p1 = p2;
+        }
+      }
+      double avgLen = sumLen / nbSeg;
+      if ( minLen > 1e-5 * avgLen )
+        tol = 0.1 * minLen; // minLen is not degenerate
+      else
+        tol = 0.1 * avgLen;
 
-      throw SALOME_Exception("Non-manifold mesh. Only manifold mesh can be re-meshed");
+      SMESH_MeshAlgos::CoincidentFreeBorders freeBords;
+      SMESH_MeshAlgos::FindCoincidentFreeBorders( *myMeshDS, tol, freeBords );
+      if ( !freeBords._coincidentGroups.empty() )
+      {
+        const char* text = "Can't re-meshed a mesh with coincident free edges";
+        SMESH_BadInputElements* error =
+          new SMESH_BadInputElements( myMeshDS, COMPERR_BAD_INPUT_MESH, text );
+        for ( size_t i = 0; i < freeBords._borders.size(); ++i )
+          error->myBadElements.insert( error->myBadElements.end(),
+                                       freeBords._borders[i].begin(),
+                                       freeBords._borders[i].end() );
+        theMesh.GetSubMesh( theMesh.GetShapeToMesh() )->GetComputeError().reset( error );
+        throw SALOME_Exception( text );
+      }
     }
 
     // fill holes
@@ -142,9 +188,10 @@ namespace
         myInHolePos[i] = ( SMESH_NodeXYZ( newFaces[0]->GetNode(0)) +
                            SMESH_NodeXYZ( newFaces[0]->GetNode(1)) +
                            SMESH_NodeXYZ( newFaces[0]->GetNode(2)) ) / 3.;
+        myCapElems.insert( myCapElems.end(), newFaces.begin(), newFaces.end() );
         // unmark to be able to remove them if meshing is canceled
-        for ( size_t iF = 0; iF < newFaces.size(); ++iF )
-          newFaces[iF]->setIsMarked( false );
+        // for ( size_t iF = 0; iF < newFaces.size(); ++iF )
+        //   newFaces[iF]->setIsMarked( false );
       }
     }
     // fix orientation
@@ -172,6 +219,8 @@ namespace
 
   void HoleFiller::AddHoleBorders( Ng_STL_Geometry * ngStlGeo )
   {
+    nglib::readedges.SetSize(0);
+
     for ( size_t i = 0; i < myHole.size(); ++i )
       for ( size_t iP = 1; iP < myHole[i].size(); ++iP )
       {
@@ -191,6 +240,13 @@ namespace
     if ( myMeshDS->NbNodes() < 3 )
       return;
 
+    if ( !myCapElems.empty() ) // old mesh not removed; simply remove myCapElems
+    {
+      for ( size_t i = 0; i < myCapElems.size(); ++i )
+        myMeshDS->RemoveFreeElement( myCapElems[i], /*sm=*/0 );
+      return;
+    }
+
     bool hasOrphanNodes = true;
 
     const double tol = Max( 1e-3 * netgen::mparam.minh, Precision::Confusion() );
@@ -413,8 +469,8 @@ namespace
  */
 //=============================================================================
 
-NETGENPlugin_Remesher_2D::NETGENPlugin_Remesher_2D(int hypId, int studyId, SMESH_Gen* gen)
-  : SMESH_2D_Algo(hypId, studyId, gen)
+NETGENPlugin_Remesher_2D::NETGENPlugin_Remesher_2D(int hypId, SMESH_Gen* gen)
+  : SMESH_2D_Algo(hypId, gen)
 {
   _name = "NETGEN_Remesher_2D";
   _shapeType = (1 << TopAbs_FACE); // 1 bit /shape type
@@ -541,6 +597,12 @@ bool NETGENPlugin_Remesher_2D::Compute(SMESH_Mesh&         theMesh,
     netgen::mparam.minh = netgen::mparam.maxh;
   }
 
+  // TODO: expose stlparam.resth* to the user
+  // netgen::stlparam.resthcloseedgeenable = 0; // Restrict H due to close edges
+  // netgen::stlparam.resthlinelengthenable = 0; // Restrict H due to line-length
+  // netgen::stlparam.resthatlasenable = 0;
+  // //netgen::stlparam.resthchartdistenable = 0;
+
   double h = netgen::mparam.maxh;
   ngMesh->SetGlobalH( h );
   ngMesh->SetMinimalH( netgen::mparam.minh );
@@ -572,6 +634,7 @@ bool NETGENPlugin_Remesher_2D::Compute(SMESH_Mesh&         theMesh,
     return error( "Error in Surface Meshing" );
 
   // remove existing mesh
+  holeFiller.ClearCapElements();
   SMDS_ElemIteratorPtr eIt = meshDS->elementsIterator();
   while ( eIt->more() )
     meshDS->RemoveFreeElement( eIt->next(), /*sm=*/0 );