Salome HOME
typo-fix by Kunda
[plugins/netgenplugin.git] / src / NETGENPlugin / NETGENPlugin_NETGEN_2D_ONLY.cxx
index 9ef97d765f06d42ed083bae280697fcdcb191890..5f2b88fc9a0c5b8804e5c248a086b72f48b37cca 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2007-2014  CEA/DEN, EDF R&D, OPEN CASCADE
+// Copyright (C) 2007-2016  CEA/DEN, EDF R&D, OPEN CASCADE
 //
 // This library is free software; you can redistribute it and/or
 // modify it under the terms of the GNU Lesser General Public
@@ -67,30 +67,33 @@ namespace netgen {
   extern int OCCGenerateMesh (OCCGeometry&, Mesh*&, MeshingParameters&, int, int);
 #else
   extern int OCCGenerateMesh (OCCGeometry&, Mesh*&, int, int, char*);
+#endif
+#if defined(NETGEN_V5) && defined(WIN32)
+  DLL_HEADER 
 #endif
   extern MeshingParameters mparam;
+  extern void OCCSetLocalMeshSize(OCCGeometry & geom, Mesh & mesh);
 }
 
 using namespace std;
 using namespace netgen;
 using namespace nglib;
 
-//#define DUMP_SEGMENTS
-
 //=============================================================================
 /*!
  *  
  */
 //=============================================================================
 
-NETGENPlugin_NETGEN_2D_ONLY::NETGENPlugin_NETGEN_2D_ONLY(int hypId, int studyId,
+NETGENPlugin_NETGEN_2D_ONLY::NETGENPlugin_NETGEN_2D_ONLY(int        hypId,
+                                                         int        studyId,
                                                          SMESH_Gen* gen)
   : SMESH_2D_Algo(hypId, studyId, gen)
 {
-  MESSAGE("NETGENPlugin_NETGEN_2D_ONLY::NETGENPlugin_NETGEN_2D_ONLY");
   _name = "NETGEN_2D_ONLY";
   
   _shapeType = (1 << TopAbs_FACE);// 1 bit /shape type
+  _onlyUnaryInput = false; // treat all FACEs at once
 
   _compatibleHypothesis.push_back("MaxElementArea");
   _compatibleHypothesis.push_back("LengthFromEdges");
@@ -98,10 +101,10 @@ NETGENPlugin_NETGEN_2D_ONLY::NETGENPlugin_NETGEN_2D_ONLY(int hypId, int studyId,
   _compatibleHypothesis.push_back("NETGEN_Parameters_2D");
   _compatibleHypothesis.push_back("ViscousLayers2D");
 
-  _hypMaxElementArea = 0;
-  _hypLengthFromEdges = 0;
+  _hypMaxElementArea       = 0;
+  _hypLengthFromEdges      = 0;
   _hypQuadranglePreference = 0;
-  _hypParameters = 0;
+  _hypParameters           = 0;
 }
 
 //=============================================================================
@@ -112,7 +115,7 @@ NETGENPlugin_NETGEN_2D_ONLY::NETGENPlugin_NETGEN_2D_ONLY(int hypId, int studyId,
 
 NETGENPlugin_NETGEN_2D_ONLY::~NETGENPlugin_NETGEN_2D_ONLY()
 {
-  MESSAGE("NETGENPlugin_NETGEN_2D_ONLY::~NETGENPlugin_NETGEN_2D_ONLY");
+  //MESSAGE("NETGENPlugin_NETGEN_2D_ONLY::~NETGENPlugin_NETGEN_2D_ONLY");
 }
 
 //=============================================================================
@@ -141,6 +144,7 @@ bool NETGENPlugin_NETGEN_2D_ONLY::CheckHypothesis (SMESH_Mesh&         aMesh,
 
   aStatus = HYP_MISSING;
 
+  bool hasVL = false;
   list<const SMESHDS_Hypothesis*>::const_iterator ith;
   for (ith = hyps.begin(); ith != hyps.end(); ++ith )
   {
@@ -157,7 +161,7 @@ bool NETGENPlugin_NETGEN_2D_ONLY::CheckHypothesis (SMESH_Mesh&         aMesh,
     else if ( hypName == "NETGEN_Parameters_2D" )
       _hypParameters = static_cast<const NETGENPlugin_Hypothesis_2D*>(hyp);
     else if ( hypName == StdMeshers_ViscousLayers2D::GetHypType() )
-      continue;
+      hasVL = true;
     else {
       aStatus = HYP_INCOMPATIBLE;
       return false;
@@ -166,13 +170,59 @@ bool NETGENPlugin_NETGEN_2D_ONLY::CheckHypothesis (SMESH_Mesh&         aMesh,
 
   int nbHyps = bool(_hypMaxElementArea) + bool(_hypLengthFromEdges) + bool(_hypParameters );
   if ( nbHyps > 1 )
-    aStatus = HYP_CONCURENT;
+    aStatus = HYP_CONCURRENT;
+  else if ( hasVL )
+    error( StdMeshers_ViscousLayers2D::CheckHypothesis( aMesh, aShape, aStatus ));
   else
     aStatus = HYP_OK;
 
+  if ( aStatus == HYP_OK && _hypParameters && _hypQuadranglePreference )
+  {
+    aStatus = HYP_INCOMPAT_HYPS;
+    return error(SMESH_Comment("\"") << _hypQuadranglePreference->GetName()
+                 << "\" and \"" << _hypParameters->GetName()
+                 << "\" are incompatible hypotheses");
+  }
+
   return ( aStatus == HYP_OK );
 }
 
+// namespace
+// {
+//   void limitSize( netgen::Mesh* ngMesh,
+//                   const double  maxh )
+//   {
+//     // get bnd box
+//     netgen::Point3d pmin, pmax;
+//     ngMesh->GetBox( pmin, pmax, 0 );
+//     const double dx = pmax.X() - pmin.X();
+//     const double dy = pmax.Y() - pmin.Y();
+//     const double dz = pmax.Z() - pmin.Z();
+
+//     const int nbX = Max( 2, int( dx / maxh * 3 ));
+//     const int nbY = Max( 2, int( dy / maxh * 3 ));
+//     const int nbZ = Max( 2, int( dz / maxh * 3 ));
+
+//     if ( ! & ngMesh->LocalHFunction() )
+//       ngMesh->SetLocalH( pmin, pmax, 0.1 );
+
+//     netgen::Point3d p;
+//     for ( int i = 0; i <= nbX; ++i )
+//     {
+//       p.X() = pmin.X() +  i * dx / nbX;
+//       for ( int j = 0; j <= nbY; ++j )
+//       {
+//         p.Y() = pmin.Y() +  j * dy / nbY;
+//         for ( int k = 0; k <= nbZ; ++k )
+//         {
+//           p.Z() = pmin.Z() +  k * dz / nbZ;
+//           ngMesh->RestrictLocalH( p, maxh );
+//         }
+//       }
+//     }
+//   }
+// }
+
 //=============================================================================
 /*!
  *Here we are going to use the NETGEN mesher
@@ -183,297 +233,372 @@ bool NETGENPlugin_NETGEN_2D_ONLY::Compute(SMESH_Mesh&         aMesh,
                                           const TopoDS_Shape& aShape)
 {
   netgen::multithread.terminate = 0;
-  netgen::multithread.task = "Surface meshing";
+  //netgen::multithread.task = "Surface meshing";
 
   SMESHDS_Mesh* meshDS = aMesh.GetMeshDS();
-  int faceID = meshDS->ShapeToIndex( aShape );
-
   SMESH_MesherHelper helper(aMesh);
-  _quadraticMesh = helper.IsQuadraticSubMesh(aShape);
   helper.SetElementsOnShape( true );
-  const bool ignoreMediumNodes = _quadraticMesh;
-
-  // build viscous layers if required
-  TopoDS_Face F = TopoDS::Face( aShape/*.Oriented( TopAbs_FORWARD )*/);
-  if ( F.Orientation() != TopAbs_FORWARD &&
-       F.Orientation() != TopAbs_REVERSED )
-    F.Orientation( TopAbs_FORWARD ); // avoid pb with TopAbs_INTERNAL
-  SMESH_ProxyMesh::Ptr proxyMesh = StdMeshers_ViscousLayers2D::Compute( aMesh, F );
-  if ( !proxyMesh )
-    return false;
 
-  // ------------------------
-  // get all edges of a face
-  // ------------------------
-  TError problem;
-  TSideVector wires =
-    StdMeshers_FaceSide::GetFaceWires( F, aMesh, ignoreMediumNodes, problem, proxyMesh );
-  if ( problem && !problem->IsOK() )
-    return error( problem );
-  int nbWires = wires.size();
-  if ( nbWires == 0 )
-    return error( "Problem in StdMeshers_FaceSide::GetFaceWires()");
-  if ( wires[0]->NbSegments() < 3 ) // ex: a circle with 2 segments
-    return error(COMPERR_BAD_INPUT_MESH,
-                 SMESH_Comment("Too few segments: ")<<wires[0]->NbSegments());
-
-  // --------------------
-  // compute edge length
-  // --------------------
+  NETGENPlugin_NetgenLibWrapper ngLib;
+  ngLib._isComputeOk = false;
+
+  netgen::Mesh   ngMeshNoLocSize;
+  netgen::Mesh * ngMeshes[2] = { (netgen::Mesh*) ngLib._ngMesh,  & ngMeshNoLocSize };
+  netgen::OCCGeometry occgeoComm;
+
+  // min / max sizes are set as follows:
+  // if ( _hypParameters )
+  //    min and max are defined by the user
+  // else if ( _hypLengthFromEdges )
+  //    min = aMesher.GetDefaultMinSize()
+  //    max = average segment len of a FACE
+  // else if ( _hypMaxElementArea )
+  //    min = aMesher.GetDefaultMinSize()
+  //    max = f( _hypMaxElementArea )
+  // else
+  //    min = aMesher.GetDefaultMinSize()
+  //    max = max segment len of a FACE
 
   NETGENPlugin_Mesher aMesher( &aMesh, aShape, /*isVolume=*/false);
-  netgen::OCCGeometry occgeo;
-  aMesher.PrepareOCCgeometry( occgeo, F, aMesh );
-  occgeo.fmap.Clear(); // face can be reversed, which is wrong in this case (issue 19978)
-  occgeo.fmap.Add( F );
-
-  if ( _hypParameters )
+  aMesher.SetParameters( _hypParameters ); // _hypParameters -> netgen::mparam
+  const bool toOptimize = _hypParameters ? _hypParameters->GetOptimize() : true;
+  if ( _hypMaxElementArea )
   {
-    aMesher.SetParameters(_hypParameters);
+    netgen::mparam.maxh = sqrt( 2. * _hypMaxElementArea->GetMaxArea() / sqrt(3.0) );
   }
-  else
+  if ( _hypQuadranglePreference )
+    netgen::mparam.quad = true;
+
+  // local size is common for all FACEs in aShape?
+  const bool isCommonLocalSize = ( !_hypLengthFromEdges && !_hypMaxElementArea && netgen::mparam.uselocalh );
+  const bool isDefaultHyp = ( !_hypLengthFromEdges && !_hypMaxElementArea && !_hypParameters );
+
+  if ( isCommonLocalSize ) // compute common local size in ngMeshes[0]
   {
-    double edgeLength = 0;
-    if (_hypLengthFromEdges /*|| (!_hypLengthFromEdges && !_hypMaxElementArea)*/)
-    {
-      int nbSegments = 0;
-      for ( int iW = 0; iW < nbWires; ++iW )
-      {
-        edgeLength += wires[ iW ]->Length();
-        nbSegments += wires[ iW ]->NbSegments();
-      }
-      if ( nbSegments )
-        edgeLength /= nbSegments;
-    }
-    else if ( _hypMaxElementArea )
+    //list< SMESH_subMesh* > meshedSM[4]; --> all sub-shapes are added to occgeoComm
+    aMesher.PrepareOCCgeometry( occgeoComm, aShape, aMesh );//, meshedSM );
+
+    // local size set at MESHCONST_ANALYSE step depends on
+    // minh, face_maxh, grading and curvaturesafety; find minh if not set by the user
+    if ( !_hypParameters || netgen::mparam.minh < DBL_MIN )
     {
-      double maxArea = _hypMaxElementArea->GetMaxArea();
-      edgeLength = sqrt(2. * maxArea/sqrt(3.0));
+      if ( !_hypParameters )
+        netgen::mparam.maxh = occgeoComm.GetBoundingBox().Diam() / 3.;
+      netgen::mparam.minh = aMesher.GetDefaultMinSize( aShape, netgen::mparam.maxh );
     }
-    else
+    // set local size depending on curvature and NOT closeness of EDGEs
+    netgen::occparam.resthcloseedgeenable = false;
+    //netgen::occparam.resthcloseedgefac = 1.0 + netgen::mparam.grading;
+    occgeoComm.face_maxh = netgen::mparam.maxh;
+    netgen::OCCSetLocalMeshSize( occgeoComm, *ngMeshes[0] );
+    occgeoComm.emap.Clear();
+    occgeoComm.vmap.Clear();
+
+    // set local size according to size of existing segments
+    const double factor = netgen::occparam.resthcloseedgefac;
+    TopTools_IndexedMapOfShape edgeMap;
+    TopExp::MapShapes( aMesh.GetShapeToMesh(), TopAbs_EDGE, edgeMap );
+    for ( int iE = 1; iE <= edgeMap.Extent(); ++iE )
     {
-      // set edgeLength by a longest segment
-      double maxSeg2 = 0;
-      for ( int iW = 0; iW < nbWires; ++iW )
+      const TopoDS_Shape& edge = edgeMap( iE );
+      if ( SMESH_Algo::isDegenerated( TopoDS::Edge( edge )))
+        continue;
+      SMESHDS_SubMesh* smDS = meshDS->MeshElements( edge );
+      if ( !smDS ) continue;
+      SMDS_ElemIteratorPtr segIt = smDS->GetElements();
+      while ( segIt->more() )
       {
-        const UVPtStructVec& points = wires[ iW ]->GetUVPtStruct();
-        gp_Pnt pPrev = SMESH_TNodeXYZ( points[0].node );
-        for ( size_t i = 1; i < points.size(); ++i )
-        {
-          gp_Pnt p = SMESH_TNodeXYZ( points[i].node );
-          maxSeg2 = Max( maxSeg2, p.SquareDistance( pPrev ));
-          pPrev = p;
-        }
+        const SMDS_MeshElement* seg = segIt->next();
+        SMESH_TNodeXYZ n1 = seg->GetNode(0);
+        SMESH_TNodeXYZ n2 = seg->GetNode(1);
+        gp_XYZ p = 0.5 * ( n1 + n2 );
+        netgen::Point3d pi(p.X(), p.Y(), p.Z());
+        ngMeshes[0]->RestrictLocalH( pi, factor * ( n1 - n2 ).Modulus() );
       }
-      edgeLength = sqrt( maxSeg2 ) * 1.05;
     }
-    if ( edgeLength < DBL_MIN )
-      edgeLength = occgeo.GetBoundingBox().Diam();
 
-    netgen::mparam.maxh = edgeLength;
-    netgen::mparam.minh = aMesher.GetDefaultMinSize( aShape, netgen::mparam.maxh );
-    netgen::mparam.quad = _hypQuadranglePreference ? 1 : 0;
-    netgen::mparam.grading = 0.4; // Moderate fineness by default
+    // set local size defined on shapes
+    aMesher.SetLocalSize( occgeoComm, *ngMeshes[0] );
+    aMesher.SetLocalSizeForChordalError( occgeoComm, *ngMeshes[0] );
+    try {
+      ngMeshes[0]->LoadLocalMeshSize( mparam.meshsizefilename );
+    } catch (NgException & ex) {
+      return error( COMPERR_BAD_PARMETERS, ex.What() );
+    }
   }
-  occgeo.face_maxh = netgen::mparam.maxh;
-
-  // -------------------------
-  // Make input netgen mesh
-  // -------------------------
+  netgen::mparam.uselocalh = toOptimize; // restore as it is used at surface optimization
 
-  // MESHCONST_ANALYSE step may lead to a failure, so we make an attempt
-  // w/o MESHCONST_ANALYSE at the second loop
-  int err = 1;
-  int iLoop = netgen::mparam.uselocalh ? 0 : 1; // uselocalh depends on 
-  for ( ; iLoop < 2; iLoop++ )
-  {
-    bool isMESHCONST_ANALYSE = false;
-    InitComputeError();
+  // ==================
+  // Loop on all FACEs
+  // ==================
 
-    NETGENPlugin_NetgenLibWrapper ngLib;
-    netgen::Mesh * ngMesh = (netgen::Mesh*) ngLib._ngMesh;
-    ngLib._isComputeOk = false;
+  vector< const SMDS_MeshNode* > nodeVec;
 
-#ifndef NETGEN_V5
-    char *optstr = 0;
-#endif
-    int startWith = MESHCONST_ANALYSE;
-    int endWith   = MESHCONST_ANALYSE;
+  TopExp_Explorer fExp( aShape, TopAbs_FACE );
+  for ( int iF = 0; fExp.More(); fExp.Next(), ++iF )
+  {
+    TopoDS_Face F = TopoDS::Face( fExp.Current() /*.Oriented( TopAbs_FORWARD )*/);
+    int    faceID = meshDS->ShapeToIndex( F );
+    SMESH_ComputeErrorPtr& faceErr = aMesh.GetSubMesh( F )->GetComputeError();
+
+    _quadraticMesh = helper.IsQuadraticSubMesh( F );
+    const bool ignoreMediumNodes = _quadraticMesh;
+
+    // build viscous layers if required
+    if ( F.Orientation() != TopAbs_FORWARD &&
+         F.Orientation() != TopAbs_REVERSED )
+      F.Orientation( TopAbs_FORWARD ); // avoid pb with TopAbs_INTERNAL
+    SMESH_ProxyMesh::Ptr proxyMesh = StdMeshers_ViscousLayers2D::Compute( aMesh, F );
+    if ( !proxyMesh )
+      continue;
 
-    if ( !_hypLengthFromEdges && !_hypMaxElementArea && iLoop == 0 )
+    // ------------------------
+    // get all EDGEs of a FACE
+    // ------------------------
+    TSideVector wires =
+      StdMeshers_FaceSide::GetFaceWires( F, aMesh, ignoreMediumNodes, faceErr, &helper, proxyMesh );
+    if ( faceErr && !faceErr->IsOK() )
+      continue;
+    int nbWires = wires.size();
+    if ( nbWires == 0 )
     {
-      isMESHCONST_ANALYSE = true;
-#ifdef NETGEN_V5
-      err = netgen::OCCGenerateMesh(occgeo, ngMesh, netgen::mparam, startWith, endWith);
-#else
-      err = netgen::OCCGenerateMesh(occgeo, ngMesh, startWith, endWith, optstr);
-#endif
-      ngLib._ngMesh = 0;
-      ngLib.setMesh(( nglib::Ng_Mesh*) ngMesh );
+      faceErr.reset
+        ( new SMESH_ComputeError
+          ( COMPERR_ALGO_FAILED, "Problem in StdMeshers_FaceSide::GetFaceWires()" ));
+      continue;
     }
-    else
+    if ( wires[0]->NbSegments() < 3 ) // ex: a circle with 2 segments
     {
-      Box<3> bb = occgeo.GetBoundingBox();
-      bb.Increase (bb.Diam()/10);
-      ngMesh->SetLocalH (bb.PMin(), bb.PMax(), netgen::mparam.grading);
-      ngMesh->SetGlobalH (netgen::mparam.maxh);
+      faceErr.reset
+        ( new SMESH_ComputeError
+          ( COMPERR_BAD_INPUT_MESH, SMESH_Comment("Too few segments: ")<<wires[0]->NbSegments()) );
+      continue;
     }
-    //cerr << "max " << netgen::mparam.maxh << " min " << netgen::mparam.minh << endl;
 
-    vector< const SMDS_MeshNode* > nodeVec;
-    problem = aMesher.AddSegmentsToMesh( *ngMesh, occgeo, wires, helper, nodeVec );
-    if ( problem && !problem->IsOK() )
-      return error( problem );
+    // ----------------------
+    // compute maxh of a FACE
+    // ----------------------
 
-    if ( iLoop == 0 )
+    if ( !_hypParameters )
     {
-      // limit element size near existing segments
-      TopTools_IndexedMapOfShape edgeMap;
-      PShapeIteratorPtr solidIt = helper.GetAncestors( F, aMesh, TopAbs_SOLID );
-      while ( const TopoDS_Shape* solid = solidIt->next() )
+      double edgeLength = 0;
+      if (_hypLengthFromEdges )
+      {
+        // compute edgeLength as an average segment length
+        int nbSegments = 0;
+        for ( int iW = 0; iW < nbWires; ++iW )
+        {
+          edgeLength += wires[ iW ]->Length();
+          nbSegments += wires[ iW ]->NbSegments();
+        }
+        if ( nbSegments )
+          edgeLength /= nbSegments;
+        netgen::mparam.maxh = edgeLength;
+      }
+      else if ( isDefaultHyp )
       {
-        TopExp_Explorer eExp( *solid, TopAbs_EDGE );
-        for ( ; eExp.More(); eExp.Next() )
+        // set edgeLength by a longest segment
+        double maxSeg2 = 0;
+        for ( int iW = 0; iW < nbWires; ++iW )
         {
-          const TopoDS_Shape& edge = eExp.Current();
-          if (( SMESH_Algo::isDegenerated( TopoDS::Edge( edge ))) ||
-              ( helper.IsSubShape( edge, aShape )) ||
-              ( !edgeMap.Add( edge )))
-            continue;
-          SMESHDS_SubMesh* smDS = aMesh.GetMeshDS()->MeshElements( edge );
-          if ( !smDS ) continue;
-          SMDS_ElemIteratorPtr segIt = smDS->GetElements();
-          while ( segIt->more() )
+          const UVPtStructVec& points = wires[ iW ]->GetUVPtStruct();
+          if ( points.empty() )
+            return error( COMPERR_BAD_INPUT_MESH );
+          gp_Pnt pPrev = SMESH_TNodeXYZ( points[0].node );
+          for ( size_t i = 1; i < points.size(); ++i )
           {
-            const SMDS_MeshElement* seg = segIt->next();
-            SMESH_TNodeXYZ n1 = seg->GetNode(0);
-            SMESH_TNodeXYZ n2 = seg->GetNode(1);
-            gp_XYZ p = 0.5 * ( n1 + n2 );
-            netgen::Point3d pi(p.X(), p.Y(), p.Z());
-            ngMesh->RestrictLocalH( pi, Max(( n1 - n2 ).Modulus(), netgen::mparam.minh ));
+            gp_Pnt p = SMESH_TNodeXYZ( points[i].node );
+            maxSeg2 = Max( maxSeg2, p.SquareDistance( pPrev ));
+            pPrev = p;
           }
         }
+        edgeLength = sqrt( maxSeg2 ) * 1.05;
+        netgen::mparam.maxh = edgeLength;
+      }
+      if ( netgen::mparam.maxh < DBL_MIN )
+        netgen::mparam.maxh = occgeoComm.GetBoundingBox().Diam();
+
+      if ( !isCommonLocalSize )
+      {
+        netgen::mparam.minh = aMesher.GetDefaultMinSize( F, netgen::mparam.maxh );
       }
     }
 
+    // prepare occgeom
+    netgen::OCCGeometry occgeom;
+    occgeom.shape = F;
+    occgeom.fmap.Add( F );
+    occgeom.CalcBoundingBox();
+    occgeom.facemeshstatus.SetSize(1);
+    occgeom.facemeshstatus = 0;
+    occgeom.face_maxh_modified.SetSize(1);
+    occgeom.face_maxh_modified = 0;
+    occgeom.face_maxh.SetSize(1);
+    occgeom.face_maxh = netgen::mparam.maxh;
+
     // -------------------------
-    // Generate surface mesh
+    // Fill netgen mesh
     // -------------------------
 
-    startWith = MESHCONST_MESHSURFACE;
-    endWith   = MESHCONST_OPTSURFACE;
+    // MESHCONST_ANALYSE step may lead to a failure, so we make an attempt
+    // w/o MESHCONST_ANALYSE at the second loop
+    int err = 0;
+    enum { LOC_SIZE, NO_LOC_SIZE };
+    int iLoop = isCommonLocalSize ? 0 : 1;
+    for ( ; iLoop < 2; iLoop++ )
+    {
+      //bool isMESHCONST_ANALYSE = false;
+      InitComputeError();
 
-    try {
-      OCC_CATCH_SIGNALS;
+      netgen::Mesh * ngMesh = ngMeshes[ iLoop ];
+      ngMesh->DeleteMesh();
 
-#ifdef NETGEN_V5
-      err = netgen::OCCGenerateMesh(occgeo, ngMesh, netgen::mparam, startWith, endWith);
-#else
-      err = netgen::OCCGenerateMesh(occgeo, ngMesh, startWith, endWith, optstr);
-#endif
-      if(netgen::multithread.terminate)
-        return false;
-      if ( err )
-        error(SMESH_Comment("Error in netgen::OCCGenerateMesh() at ") << netgen::multithread.task);
-    }
-    catch (Standard_Failure& ex)
-    {
-      SMESH_Comment str("Exception in  netgen::OCCGenerateMesh()");
-      str << " at " << netgen::multithread.task
-          << ": " << ex.DynamicType()->Name();
-      if ( ex.GetMessageString() && strlen( ex.GetMessageString() ))
-        str << ": " << ex.GetMessageString();
-      error(str);
-      err = 1;
-    }
-    catch (...) {
-      SMESH_Comment str("Exception in  netgen::OCCGenerateMesh()");
-      str << " at " << netgen::multithread.task;
-      error(str);
-      err = 1;
-    }
-    if ( err /*&& !isMESHCONST_ANALYSE*/ && iLoop == 0 )
-    {
-      netgen::mparam.minh = netgen::mparam.maxh;
-      netgen::mparam.maxh = 0;
-      for ( int iW = 0; iW < wires.size(); ++iW )
+      if ( iLoop == NO_LOC_SIZE )
       {
-        StdMeshers_FaceSidePtr wire = wires[ iW ];
-        const vector<UVPtStruct>& uvPtVec = wire->GetUVPtStruct();
-        for ( size_t iP = 1; iP < uvPtVec.size(); ++iP )
-        {
-          SMESH_TNodeXYZ   p( uvPtVec[ iP ].node );
-          netgen::Point3d np( p.X(),p.Y(),p.Z());
-          double segLen = p.Distance( uvPtVec[ iP-1 ].node );
-          double   size = ngMesh->GetH( np );
-          netgen::mparam.minh = Min( netgen::mparam.minh, size );
-          netgen::mparam.maxh = Max( netgen::mparam.maxh, segLen );
+        ngMesh->SetGlobalH ( mparam.maxh );
+        ngMesh->SetMinimalH( mparam.minh );
+        Box<3> bb = occgeom.GetBoundingBox();
+        bb.Increase (bb.Diam()/10);
+        ngMesh->SetLocalH (bb.PMin(), bb.PMax(), mparam.grading);
+        aMesher.SetLocalSize( occgeom, *ngMesh );
+        aMesher.SetLocalSizeForChordalError( occgeoComm, *ngMesh );
+        try {
+          ngMesh->LoadLocalMeshSize( mparam.meshsizefilename );
+        } catch (NgException & ex) {
+          return error( COMPERR_BAD_PARMETERS, ex.What() );
         }
       }
-      //cerr << "min " << netgen::mparam.minh << " max " << netgen::mparam.maxh << endl;
-      netgen::mparam.minh *= 0.9;
-      netgen::mparam.maxh *= 1.1;
 
-      continue;
-    }
+      nodeVec.clear();
+      faceErr = aMesher.AddSegmentsToMesh( *ngMesh, occgeom, wires, helper, nodeVec,
+                                           /*overrideMinH=*/!_hypParameters);
+      if ( faceErr && !faceErr->IsOK() )
+        break;
 
-    // ----------------------------------------------------
-    // Fill the SMESHDS with the generated nodes and faces
-    // ----------------------------------------------------
+      //if ( !isCommonLocalSize )
+      //limitSize( ngMesh, mparam.maxh * 0.8);
 
-    int nbNodes = ngMesh->GetNP();
-    int nbFaces = ngMesh->GetNSE();
+      // -------------------------
+      // Generate surface mesh
+      // -------------------------
 
-    int nbInputNodes = nodeVec.size()-1;
-    nodeVec.resize( nbNodes+1, 0 );
+      const int startWith = MESHCONST_MESHSURFACE;
+      const int endWith   = toOptimize ? MESHCONST_OPTSURFACE : MESHCONST_MESHSURFACE;
 
-    // add nodes
-    for ( int ngID = nbInputNodes + 1; ngID <= nbNodes; ++ngID )
-    {
-      const MeshPoint& ngPoint = ngMesh->Point( ngID );
-      SMDS_MeshNode * node = meshDS->AddNode(ngPoint(0), ngPoint(1), ngPoint(2));
-      nodeVec[ ngID ] = node;
-    }
+      SMESH_Comment str;
+      try {
+        OCC_CATCH_SIGNALS;
 
-    // create faces
-    const bool reverse = false; //( aShape.Orientation() == TopAbs_REVERSED );
-    int i,j;
-    for ( i = 1; i <= nbFaces ; ++i )
-    {
-      const Element2d& elem = ngMesh->SurfaceElement(i);
-      vector<const SMDS_MeshNode*> nodes( elem.GetNP() );
-      for (j=1; j <= elem.GetNP(); ++j)
+#ifdef NETGEN_V5
+        err = netgen::OCCGenerateMesh(occgeom, ngMesh, netgen::mparam, startWith, endWith);
+#else
+        char *optstr = 0;
+        err = netgen::OCCGenerateMesh(occgeom, ngMesh, startWith, endWith, optstr);
+#endif
+        if ( netgen::multithread.terminate )
+          return false;
+        if ( err )
+          str << "Error in netgen::OCCGenerateMesh() at " << netgen::multithread.task;
+      }
+      catch (Standard_Failure& ex)
       {
-        int pind = elem.PNum(j);
-        if ( pind < 1 )
+        err = 1;
+        str << "Exception in  netgen::OCCGenerateMesh()"
+            << " at " << netgen::multithread.task
+            << ": " << ex.DynamicType()->Name();
+        if ( ex.GetMessageString() && strlen( ex.GetMessageString() ))
+          str << ": " << ex.GetMessageString();
+      }
+      catch (...) {
+        err = 1;
+        str << "Exception in  netgen::OCCGenerateMesh()"
+            << " at " << netgen::multithread.task;
+      }
+      if ( err )
+      {
+        if ( aMesher.FixFaceMesh( occgeom, *ngMesh, 1 ))
           break;
-        const SMDS_MeshNode* node = nodeVec[ pind ];
-        if ( reverse )
-          nodes[ nodes.size()-j ] = node;
+        if ( iLoop == LOC_SIZE )
+        {
+          netgen::mparam.minh = netgen::mparam.maxh;
+          netgen::mparam.maxh = 0;
+          for ( size_t iW = 0; iW < wires.size(); ++iW )
+          {
+            StdMeshers_FaceSidePtr wire = wires[ iW ];
+            const vector<UVPtStruct>& uvPtVec = wire->GetUVPtStruct();
+            for ( size_t iP = 1; iP < uvPtVec.size(); ++iP )
+            {
+              SMESH_TNodeXYZ   p( uvPtVec[ iP ].node );
+              netgen::Point3d np( p.X(),p.Y(),p.Z());
+              double segLen = p.Distance( uvPtVec[ iP-1 ].node );
+              double   size = ngMesh->GetH( np );
+              netgen::mparam.minh = Min( netgen::mparam.minh, size );
+              netgen::mparam.maxh = Max( netgen::mparam.maxh, segLen );
+            }
+          }
+          //cerr << "min " << netgen::mparam.minh << " max " << netgen::mparam.maxh << endl;
+          netgen::mparam.minh *= 0.9;
+          netgen::mparam.maxh *= 1.1;
+          continue;
+        }
         else
-          nodes[ j-1 ] = node;
-        if ( node->GetPosition()->GetTypeOfPosition() == SMDS_TOP_3DSPACE )
         {
-          const PointGeomInfo& pgi = elem.GeomInfoPi(j);
-          meshDS->SetNodeOnFace((SMDS_MeshNode*)node, faceID, pgi.u, pgi.v);
+          faceErr.reset( new SMESH_ComputeError( COMPERR_ALGO_FAILED, str ));
         }
       }
-      if ( j > elem.GetNP() )
+
+
+      // ----------------------------------------------------
+      // Fill the SMESHDS with the generated nodes and faces
+      // ----------------------------------------------------
+
+      int nbNodes = ngMesh->GetNP();
+      int nbFaces = ngMesh->GetNSE();
+
+      int nbInputNodes = nodeVec.size()-1;
+      nodeVec.resize( nbNodes+1, 0 );
+
+      // add nodes
+      for ( int ngID = nbInputNodes + 1; ngID <= nbNodes; ++ngID )
       {
-        SMDS_MeshFace* face = 0;
-        if ( elem.GetType() == TRIG )
-          face = helper.AddFace(nodes[0],nodes[1],nodes[2]);
-        else
-          face = helper.AddFace(nodes[0],nodes[1],nodes[2],nodes[3]);
+        const MeshPoint& ngPoint = ngMesh->Point( ngID );
+        SMDS_MeshNode * node = meshDS->AddNode(ngPoint(0), ngPoint(1), ngPoint(2));
+        nodeVec[ ngID ] = node;
       }
-    }
 
-    ngLib._isComputeOk = !err;
-    break;
+      // create faces
+      int i,j;
+      vector<const SMDS_MeshNode*> nodes;
+      for ( i = 1; i <= nbFaces ; ++i )
+      {
+        const Element2d& elem = ngMesh->SurfaceElement(i);
+        nodes.resize( elem.GetNP() );
+        for (j=1; j <= elem.GetNP(); ++j)
+        {
+          int pind = elem.PNum(j);
+          if ( pind < 1 )
+            break;
+          nodes[ j-1 ] = nodeVec[ pind ];
+          if ( nodes[ j-1 ]->GetPosition()->GetTypeOfPosition() == SMDS_TOP_3DSPACE )
+          {
+            const PointGeomInfo& pgi = elem.GeomInfoPi(j);
+            meshDS->SetNodeOnFace( nodes[ j-1 ], faceID, pgi.u, pgi.v);
+          }
+        }
+        if ( j > elem.GetNP() )
+        {
+          if ( elem.GetType() == TRIG )
+            helper.AddFace(nodes[0],nodes[1],nodes[2]);
+          else
+            helper.AddFace(nodes[0],nodes[1],nodes[2],nodes[3]);
+        }
+      }
 
-  } // two attempts
+      break;
+    } // two attempts
+  } // loop on FACEs
 
-  return !err;
+  return true;
 }
 
 void NETGENPlugin_NETGEN_2D_ONLY::CancelCompute()
@@ -490,26 +615,27 @@ void NETGENPlugin_NETGEN_2D_ONLY::CancelCompute()
 
 double NETGENPlugin_NETGEN_2D_ONLY::GetProgress() const
 {
-  const char* task1 = "Surface meshing";
-  //const char* task2 = "Optimizing surface";
-  double& progress = const_cast<NETGENPlugin_NETGEN_2D_ONLY*>( this )->_progress;
-  if ( _progressByTic < 0. &&
-       strncmp( netgen::multithread.task, task1, 3 ) == 0 )
-  {
-    progress = Min( 0.25, SMESH_Algo::GetProgressByTic() ); // [0, 0.25]
-  }
-  else //if ( strncmp( netgen::multithread.task, task2, 3 ) == 0)
-  {
-    if ( _progressByTic < 0 )
-    {
-      NETGENPlugin_NETGEN_2D_ONLY* me = (NETGENPlugin_NETGEN_2D_ONLY*) this;
-      me->_progressByTic = 0.25 / (_progressTic+1);
-    }
-    const_cast<NETGENPlugin_NETGEN_2D_ONLY*>( this )->_progressTic++;
-    progress = Max( progress, _progressByTic * _progressTic );
-  }
-  //cout << netgen::multithread.task << " " << _progressTic << endl;
-  return Min( progress, 0.99 );
+  return -1;
+  // const char* task1 = "Surface meshing";
+  // //const char* task2 = "Optimizing surface";
+  // double& progress = const_cast<NETGENPlugin_NETGEN_2D_ONLY*>( this )->_progress;
+  // if ( _progressByTic < 0. &&
+  //      strncmp( netgen::multithread.task, task1, 3 ) == 0 )
+  // {
+  //   progress = Min( 0.25, SMESH_Algo::GetProgressByTic() ); // [0, 0.25]
+  // }
+  // else //if ( strncmp( netgen::multithread.task, task2, 3 ) == 0)
+  // {
+  //   if ( _progressByTic < 0 )
+  //   {
+  //     NETGENPlugin_NETGEN_2D_ONLY* me = (NETGENPlugin_NETGEN_2D_ONLY*) this;
+  //     me->_progressByTic = 0.25 / (_progressTic+1);
+  //   }
+  //   const_cast<NETGENPlugin_NETGEN_2D_ONLY*>( this )->_progressTic++;
+  //   progress = Max( progress, _progressByTic * _progressTic );
+  // }
+  // //cout << netgen::multithread.task << " " << _progressTic << endl;
+  // return Min( progress, 0.99 );
 }
 
 //=============================================================================
@@ -559,7 +685,7 @@ bool NETGENPlugin_NETGEN_2D_ONLY::Evaluate(SMESH_Mesh& aMesh,
 
   // compute edge length
   double ELen = 0;
-  if (_hypLengthFromEdges || !_hypLengthFromEdges && !_hypMaxElementArea) {
+  if (( _hypLengthFromEdges ) || ( !_hypLengthFromEdges && !_hypMaxElementArea )) {
     if ( nb1d > 0 )
       ELen = fullLen / nb1d;
   }