0032116: Visualization - AIS_Manipulator is unusable when attaching to objects with Graphic3d_TMF_ZoomPers

Recalculation of manipulator position if attached object has zooom transform persistence as well as set its anchor point in Transformation.

src/AIS/AIS_Manipulator.cxx

@@ -398,17 +398,25 @@ void AIS_Manipulator::Attach (const Handle(AIS_ManipulatorObjectSequence)& theOb
   const Handle(AIS_InteractiveContext)& aContext = Object()->GetContext();
   if (!aContext.IsNull())
   {
-    if (!aContext->IsDisplayed (this))
+    if (!aCurObject.IsNull())
     {
-      aContext->Display (this, Standard_False);
+      Handle(Graphic3d_TransformPers) aTransPers = aCurObject->TransformPersistence();
+      if (!aTransPers.IsNull())
+      {
+        aCurObject->TransformPersistence()->SetAnchorPoint(myPosition.Location());
+      }
+    }
+    if (!aContext->IsDisplayed(this))
+    {
+      aContext->Display(this, Standard_False);
     }
     else
     {
-      aContext->Update (this, Standard_False);
-      aContext->RecomputeSelectionOnly (this);
+      aContext->Update(this, Standard_False);
+      aContext->RecomputeSelectionOnly(this);
     }
 
-    aContext->Load (this);
+    aContext->Load(this);
   }
 
   if (theOptions.EnableModes)
@@ -750,6 +758,11 @@ void AIS_Manipulator::Transform (const gp_Trsf& theTrsf)
       {
         anObj->SetLocalTransformation (theTrsf * anOldTrsf);
       }
+      Handle(Graphic3d_TransformPers) aTransPers = anObj->TransformPersistence();
+      if (!aTransPers.IsNull())
+      {
+        anObj->TransformPersistence()->SetAnchorPoint(myPosition.Location());
+      }
     }
   }
 
@@ -806,15 +819,32 @@ void AIS_Manipulator::updateTransformation()
 {
   gp_Trsf aTrsf;
 
-  if (!myIsZoomPersistentMode)
+  Handle(AIS_ManipulatorObjectSequence) anObjects = Objects();
+  Handle(AIS_InteractiveObject) anObj = Object();
+  for (AIS_ManipulatorObjectSequence::Iterator anObjIter(*anObjects); anObjIter.More(); anObjIter.Next())
   {
-    aTrsf.SetTransformation (myPosition, gp::XOY());
+    anObj = anObjIter.Value();
   }
-  else
+  if (!anObj.IsNull())
   {
-    const gp_Dir& aVDir = myPosition.Direction();
-    const gp_Dir& aXDir = myPosition.XDirection();
-    aTrsf.SetTransformation (gp_Ax2 (gp::Origin(), aVDir, aXDir), gp::XOY());
+    Handle(Graphic3d_TransformPers) aTransPers = anObj->TransformPersistence();
+    if (!aTransPers.IsNull())
+    {
+      aTrsf.SetTransformation(myPosition, gp::XOY());
+    }
+    else
+    {
+      if (!myIsZoomPersistentMode)
+      {
+        aTrsf.SetTransformation(myPosition, gp::XOY());
+      }
+      else
+      {
+        const gp_Dir& aVDir = myPosition.Direction();
+        const gp_Dir& aXDir = myPosition.XDirection();
+        aTrsf.SetTransformation(gp_Ax2(gp::Origin(), aVDir, aXDir), gp::XOY());
+      }
+    }
   }
 
   Handle(TopLoc_Datum3D) aGeomTrsf = new TopLoc_Datum3D (aTrsf);

src/BRepMesh/BRepMesh_ConeRangeSplitter.cxx

@@ -35,17 +35,13 @@ std::pair<Standard_Real, Standard_Real> BRepMesh_ConeRangeSplitter::GetSplitStep
   Standard_Real aRadius = Max(Abs(aRefR + aRangeV.first  * Sin(aSAng)),
                               Abs(aRefR + aRangeV.second * Sin(aSAng)));
 
-  //Face deflection used to determine circle subdivisions needs to match the one used previously
   Standard_Real Dv, Du = GCPnts_TangentialDeflection::ArcAngularStep(
-    aRadius, GetDFace()->GetDeflection()*0.5,
+    aRadius, GetDFace()->GetDeflection(),
     theParameters.Angle, theParameters.MinSize);
 
   const Standard_Real aDiffU = aRangeU.second - aRangeU.first;
   const Standard_Real aDiffV = aRangeV.second - aRangeV.first;
-  //compute subdivision factor acounting for the difference between the bottom and top radius of the cone
-  const Standard_Real aTopRadius = aRefR > Precision::Confusion() ? aRefR : 1.0;
-  const Standard_Real aSubDivFactor = Max(theParameters.MinSize, Min(1.0, aTopRadius > aRadius ? aRadius / aTopRadius : aTopRadius / aRadius));
-  const Standard_Real aScale = (Du * aRadius * aSubDivFactor);
+  const Standard_Real aScale = (Du * aRadius);
   const Standard_Real aRatio = Max(1., Log(aDiffV / aScale));
   const Standard_Integer nbU = (Standard_Integer)(aDiffU / Du);
   const Standard_Integer nbV = (Standard_Integer)(aDiffV / aScale / aRatio);

src/StdPrs/StdPrs_ShadedShape.cxx

@@ -210,9 +210,9 @@ namespace
         {
           const gp_Pnt2d aNode2d = aT->UVNode (aNodeIter);
           const gp_Pnt2d aTexel = (dUmax == 0.0 || dVmax == 0.0)
-            ? aNode2d
-            : gp_Pnt2d ((-theUVOrigin.X() + (theUVRepeat.X() * (aNode2d.X() - aUmin)) / dUmax) / theUVScale.X(),
-              (-theUVOrigin.Y() + (theUVRepeat.Y() * (aNode2d.Y() - aVmin)) / dVmax) / theUVScale.Y());
+                                ? aNode2d
+                                : gp_Pnt2d ((-theUVOrigin.X() + (theUVRepeat.X() * (aNode2d.X() - aUmin)) / dUmax) / theUVScale.X(),
+                                            (-theUVOrigin.Y() + (theUVRepeat.Y() * (aNode2d.Y() - aVmin)) / dVmax) / theUVScale.Y());
           anArray->AddVertex (aPoint, aNorm, aTexel);
         }
         else