Salome HOME
29.01.2014. Fix bug with point classification.
authorszy <szy@opencascade.com>
Wed, 29 Jan 2014 10:24:44 +0000 (10:24 +0000)
committerszy <szy@opencascade.com>
Wed, 29 Jan 2014 10:24:44 +0000 (10:24 +0000)
src/HYDROData/HYDROData_CalculationCase.cxx
src/HYDROData/HYDROData_SplitToZonesTool.cxx
src/HYDROData/HYDROData_Tool.cxx

index 8498732a85d611bef94a1951a8f72e4be3652e66..a9be2ccec3fff0d145f76a3fa25eae60711c6ff1 100644 (file)
 #define CALCULATION_REGIONS_PREF GetName() + "_Reg"
 #define CALCULATION_ZONES_PREF GetName() + "_Zone"
 #define CALCULATION_GROUPS_PREF GetName() + "_"
+//#define DEB_CLASS2D 1
+#ifdef DEB_CLASS2D
+#include <BRepBuilderAPI_MakeVertex.hxx>
+#endif
 
 #define EXPORT_NAME "HYDRO_" + GetName()
 
@@ -669,7 +673,16 @@ HYDROData_CalculationCase::PointClassification HYDROData_CalculationCase::GetPoi
   TopoDS_Face aZoneFace = TopoDS::Face( theZone->GetShape() );
   if ( aZoneFace.IsNull() )
     return aRes;
-  
+#ifdef DEB_CLASS2D      
+         TopoDS_Compound aCmp;
+      BRep_Builder aBB;
+      aBB.MakeCompound(aCmp);
+         aBB.Add(aCmp, aZoneFace);
+         gp_Pnt aPnt (thePoint.X(), thePoint.Y(), 0.);
+         BRepBuilderAPI_MakeVertex aMk(aPnt);
+         aBB.Add(aCmp, aMk.Vertex());
+         BRepTools::Write(aCmp, "FCL2d.brep");
+#endif  
   TopAbs_State State = HYDROData_Tool::ComputePointState(thePoint, aZoneFace);
   if (State == TopAbs_OUT)
     aRes =  POINT_OUT;
index d076706fadf98920b6266bf9f770ddc7d9932e80..2205bc0b67536a87c43b1e1af4f84bdadd8ca15f 100644 (file)
@@ -24,7 +24,7 @@
 #include <BRepBuilderAPI_FindPlane.hxx>
 
 //#define DEB_SPLIT_TO_ZONES 1
-//#define DEB_SPLIT_TO_ZONES_CHECK_PARTITION 1
+#define DEB_SPLIT_TO_ZONES_CHECK_PARTITION 1
 #if (defined (DEB_SPLIT_TO_ZONES) || defined(DEB_SPLIT_TO_ZONES_CHECK_PARTITION))
 #include <BRepTools.hxx>
 static TCollection_AsciiString fileNameBefore("BeforeTranslation");
@@ -213,8 +213,10 @@ HYDROData_SplitToZonesTool::SplitDataList
     aDM3.Bind(aSh, anInputSplitDataList.at(i).ObjectNames);
     aLS.Append(aSh);
        aBB.Add(aCmp,aSh);
+#ifdef DEB_SPLIT_TO_ZONES
     //TCollection_AsciiString aName = aNam + i + ".brep";
     //BRepTools::Write(aSh, aName.ToCString());
+#endif
   }
 #ifdef DEB_SPLIT_TO_ZONES_CHECK_PARTITION
   TCollection_AsciiString aNameBefore = fileNameBefore + ".brep";
@@ -256,13 +258,15 @@ HYDROData_SplitToZonesTool::SplitDataList
     TopTools_ListIteratorOfListOfShape it(aListOfNew);
     for(;it.More();it.Next())       
       aList.Append(it.Value());
+    /*
     // Bug in History: partition should give only modified entities! => temporary solution is used
     const TopTools_ListOfShape& aListOfGen = splitTool.Generated(anIt.Value());
-       if(!aListOfGen.IsEmpty())
+       if(!aListOfGen.IsEmpty()) {
          foundF = Standard_True;
-       it.Initialize(aListOfGen);    
-       for(;it.More();it.Next())     
-         aList.Append(it.Value());
+         it.Initialize(aListOfGen);    
+         for(;it.More();it.Next())     
+           aList.Append(it.Value());
+       }*/
     if(!foundF) // face is not modified
          aList.Append (anIt.Value());
        aDM1.Bind(anIt.Value(), aList);
@@ -283,14 +287,16 @@ HYDROData_SplitToZonesTool::SplitDataList
                  //aName = aNamM + i + j +k +".brep";
                  //BRepTools::Write(it.Value(),aName.ToCString());
                }
-               const TopTools_ListOfShape& aListG = splitTool.Generated(exp.Current());
-               if(aListG.Extent()) foundE = Standard_True;
-               it.Initialize(aListG);    
-               for(int k=1;it.More();it.Next(),k++)   {  
-             aList.Append(it.Value());
+               /* const TopTools_ListOfShape& aListG = splitTool.Generated(exp.Current());
+               if(!aListG.IsEmpty()) {
+                 foundE = Standard_True;
+                 it.Initialize(aListG);    
+                 for(int k=1;it.More();it.Next(),k++)   {  
+               aList.Append(it.Value());
                  //aName = aNamG + i + j +k +".brep";
                  //BRepTools::Write(it.Value(),aName.ToCString());
-               }
+                 }
+               }*/
                //cout << "NB_EDGE = " << aList.Extent() <<endl;
                if(!foundE) {
                  aList.Append (exp.Current());
@@ -341,8 +347,7 @@ HYDROData_SplitToZonesTool::SplitDataList
               const TopoDS_Shape& aFace = exp.Current();
                      if(!aFace.IsNull()) {
                        const TopTools_ListOfShape& aListOfNew = mkCom.Modified(aFace);
-                           //cout << "Modified: " << aListOfNew.Extent() <<endl;                       
-                const TopTools_ListOfShape& aListOfGen = mkCom.Generated(aFace);                       
+                           //cout << "Modified: " << aListOfNew.Extent() <<endl;                                                       
                            if(!aListOfNew.IsEmpty()) {
                      aDM4.Bind(aFace, aListOfNew);
                              //TCollection_AsciiString aName = aNam + i + ".brep";
@@ -350,6 +355,7 @@ HYDROData_SplitToZonesTool::SplitDataList
                                }
                                else {
                              if(!mkCom.IsDeleted(aFace)) {
+                                       const TopTools_ListOfShape& aListOfGen = mkCom.Generated(aFace);        
                                    if(!aListOfGen.IsEmpty()) {
                          aDM4.Bind(aFace, aListOfGen);                        
                                      //TCollection_AsciiString aName = aNam + i + "g.brep";
@@ -368,12 +374,12 @@ HYDROData_SplitToZonesTool::SplitDataList
                           if(!anEdge.IsNull()) {
                                 if(aView.Contains(anEdge)) continue;
                                 aView.Add(anEdge);
-                                const TopTools_ListOfShape& aListOfNewEd = mkCom.Modified(anEdge);
-                                const TopTools_ListOfShape& aListOfGenEd = mkCom.Generated(anEdge);
+                                const TopTools_ListOfShape& aListOfNewEd = mkCom.Modified(anEdge);                              
                  if(!aListOfNewEd.IsEmpty())
                       aDM4.Bind(anEdge, aListOfNewEd);
                                 else {
                                   if(!mkCom.IsDeleted(anEdge)) {
+                                        const TopTools_ListOfShape& aListOfGenEd = mkCom.Generated(anEdge);
                                         if(!aListOfGenEd.IsEmpty()) {
                                           aDM4.Bind(anEdge, aListOfGenEd);
                                         } else {
index 333205a0e371be975b631e52c75e5e0991e767f0..b058b8f31a0d59fbbb774232662c0c4061fafdbd 100644 (file)
@@ -14,8 +14,6 @@
 #include <QTextStream>
 
 #include <limits>
-#define CLASS2D 1
-#ifdef CLASS2D
 #include <gp_Pnt.hxx>
 #include <gp_Vec.hxx>
 #include <gp_Pln.hxx>
 #include <TopTools_MapOfShape.hxx>
 #include <TColgp_SequenceOfVec.hxx>
 #include <TopTools_ShapeMapHasher.hxx>
-#undef _NCollection_MapHasher
-#endif
+#include <ElSLib.hxx>
+#include <BRepAdaptor_Surface.hxx>
+#include <BRepTopAdaptor_FClass2d.hxx>
+#include <BRep_Tool.hxx>
+#include <TopAbs_State.hxx>
 static int aMaxNameId = std::numeric_limits<int>::max();
 
 void HYDROData_Tool::WriteStringsToFile( QFile&             theFile,
@@ -177,63 +178,18 @@ QString HYDROData_Tool::GenerateNameForPython( const MapOfTreatedObjects& theTre
   return aName;
 }
 //======================================================================================================
-// the Face to be: planer, whitout holes
-#ifdef CLASS2D
-TopAbs_State HYDROData_Tool::ComputePointState( const gp_XY& thePnt2d, const TopoDS_Face& theFace )
+TopAbs_State HYDROData_Tool::ComputePointState( const gp_XY& theXY, const TopoDS_Face& theFace )
 {
   TopAbs_State aState(TopAbs_UNKNOWN);
   if(theFace.IsNull()) return aState;  
-
-#ifdef CLASS2D    
-  TopoDS_Wire aWire; int nb(0);
-  TopoDS_Iterator it(theFace);
-  for(;it.More();it.Next()) {
-         aWire = TopoDS::Wire(it.Value());
-         nb++;
-  }
-  if(nb > 1 || aWire.IsNull()) return aState;
-  gp_Pln aPlane;
-  gp_Vec aNormal;
-  BRepBuilderAPI_FindPlane fndPlane (theFace, Precision::Confusion());
-  if(fndPlane.Found())
-    aPlane = fndPlane.Plane()->Pln();
-  else
-    return aState;
-  aNormal = gp_Vec(aPlane.Axis().Direction());
-  if(theFace.Orientation() == TopAbs_REVERSED) 
-    aNormal.Reverse();
-  gp_Pnt aPoint = gp_Pnt (thePnt2d.X(), thePnt2d.Y(), 0);
-  TColgp_SequenceOfVec aSeq;
-  TopTools_MapOfShape aMap;
-  it.Initialize(aWire);
-  for (;it.More(); it.Next()) {
-       const TopoDS_Vertex& aVx = TopExp::FirstVertex(TopoDS::Edge(it.Value()), Standard_True);
-       if(!aMap.Add(aVx))
-         continue;
-       const gp_Pnt& aCurPnt = BRep_Tool::Pnt(TopoDS::Vertex(aVx));
-       if(aPoint.IsEqual(aCurPnt,Precision::Confusion())) {
-      aState = TopAbs_ON;
-         return aState;
-       }
-       gp_Vec aVec (aPoint, aCurPnt);
-       aSeq.Append(aVec);
-  }
-  Standard_Real anAng(0.0);
-  for(int i = 1;i < aSeq.Length();i++) {
-    const gp_Vec& aV1 = aSeq.Value(i);
-       const gp_Vec& aV2 = aSeq.Value(i+1);
-        anAng += aV1.AngleWithRef(aV2, aNormal);
-  }
-  anAng += aSeq.Value(aSeq.Length()).AngleWithRef(aSeq.Value(1), aNormal);
-  if(abs(anAng) > Precision::Angular())
-    aState = TopAbs_IN;
-  else
-        aState = TopAbs_OUT;
-
-#else
-  BRepTopAdaptor_FClass2d aClassifier( theFace, Precision::Confusion() ); 
-  aState = aClassifier.Perform( gp_Pnt2d( thePoint ), Standard_False );
-#endif
+  Standard_Real aTol = BRep_Tool::Tolerance(theFace);
+  BRepAdaptor_Surface Ads ( theFace, Standard_False );
+  Standard_Real toluv = Min ( Ads.UResolution(aTol), Ads.VResolution(aTol) ); 
+  const gp_Pln& aPlane = Ads.Surface().Plane();
+  gp_Pnt aPnt(theXY.X(), theXY.Y(), 0.);
+  Standard_Real aU1, aV1;
+  ElSLib::Parameters(aPlane,aPnt, aU1, aV1);
+  BRepTopAdaptor_FClass2d aClassifier( theFace, toluv ); 
+  aState = aClassifier.Perform( gp_Pnt2d(aU1, aV1), Standard_False );
   return aState;
-}
-#endif
\ No newline at end of file
+}
\ No newline at end of file