From: jfa Date: Mon, 26 Jun 2023 16:56:13 +0000 (+0100) Subject: [bos #35094] [EDF] (2023-T1) X,Y,Z to U,V. Consider face tolerance. X-Git-Tag: V9_12_0a1~6 X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=e8fc8b6907dd8297fc63cc0398e2def3d99301e2;p=modules%2Fgeom.git [bos #35094] [EDF] (2023-T1) X,Y,Z to U,V. Consider face tolerance. --- diff --git a/src/GEOMImpl/GEOMImpl_IMeasureOperations.cxx b/src/GEOMImpl/GEOMImpl_IMeasureOperations.cxx index e191364bb..387302224 100644 --- a/src/GEOMImpl/GEOMImpl_IMeasureOperations.cxx +++ b/src/GEOMImpl/GEOMImpl_IMeasureOperations.cxx @@ -3032,8 +3032,8 @@ Handle(TColStd_HArray1OfReal) GEOMImpl_IMeasureOperations::XYZtoUV } // Face tolerance - Standard_Real squareTolerance = BRep_Tool::Tolerance(F); - squareTolerance = squareTolerance * squareTolerance; + Standard_Real aTol = BRep_Tool::Tolerance(F); + Standard_Real squareTolerance = aTol * aTol; // Compute parameters Handle(Geom_Surface) aSurf = BRep_Tool::Surface(F); @@ -3049,19 +3049,25 @@ Handle(TColStd_HArray1OfReal) GEOMImpl_IMeasureOperations::XYZtoUV gp_Pnt aP (theXYZlist->Value(iCLower + iP * 3), theXYZlist->Value(iCLower + iP * 3 + 1), theXYZlist->Value(iCLower + iP * 3 + 2)); - Standard_Real U, V; - gp_Pnt aPonF = GEOMUtils::ProjectPointOnFace(aP, F, U, V); - if (aP.SquareDistance(aPonF) < squareTolerance) { - if (isNormalized) { - // Normalize parameters to be in [0, 1] - U = (U - U1) / dU; - V = (V - V1) / dV; + try { // as GEOMUtils::ProjectPointOnFace can throw exceptions + Standard_Real U, V; + gp_Pnt aPonF = GEOMUtils::ProjectPointOnFace(aP, F, U, V, aTol); + if (aP.SquareDistance(aPonF) < squareTolerance) { + if (isNormalized) { + // Normalize parameters to be in [0, 1] + U = (U - U1) / dU; + V = (V - V1) / dV; + } + aRet->SetValue(iP * 2 , U); + aRet->SetValue(iP * 2 + 1, V); + } + else { + SetErrorCode("Point too far from face"); + return aRet; } - aRet->SetValue(iP * 2 , U); - aRet->SetValue(iP * 2 + 1, V); } - else { - SetErrorCode("Point too far from face"); + catch (Standard_Failure& aFail) { + SetErrorCode(aFail.GetMessageString()); return aRet; } } diff --git a/src/GEOMUtils/GEOMUtils.cxx b/src/GEOMUtils/GEOMUtils.cxx index 4abf1f3b3..35cd7f0fd 100644 --- a/src/GEOMUtils/GEOMUtils.cxx +++ b/src/GEOMUtils/GEOMUtils.cxx @@ -1032,7 +1032,8 @@ Standard_Real GEOMUtils::GetMinDistance //======================================================================= gp_Pnt GEOMUtils::ProjectPointOnFace(const gp_Pnt& thePoint, const TopoDS_Shape& theFace, - double& theU, double& theV) + double& theU, double& theV, + const double theTol) { if (theFace.IsNull() || theFace.ShapeType() != TopAbs_FACE) Standard_TypeMismatch::Raise @@ -1044,7 +1045,7 @@ gp_Pnt GEOMUtils::ProjectPointOnFace(const gp_Pnt& thePoint, BRepTools::UVBounds(aFace, U1, U2, V1, V2); // projector - Standard_Real tol = 1.e-4; + Standard_Real tol = Max(theTol, 1.e-4); GeomAPI_ProjectPointOnSurf proj; proj.Init(surface, U1, U2, V1, V2, tol); proj.Perform(thePoint); diff --git a/src/GEOMUtils/GEOMUtils.hxx b/src/GEOMUtils/GEOMUtils.hxx index 5546211fa..6fbbac9b5 100644 --- a/src/GEOMUtils/GEOMUtils.hxx +++ b/src/GEOMUtils/GEOMUtils.hxx @@ -74,13 +74,13 @@ namespace GEOMUtils * \brief Compute numerical functor for the shape. * * Resulting value can be used to sort out shapes according to some parameter. - * + * * Returns a pair of two values (dist, functor) where * - \a dist is a some value that is computed according to the center of mass of given shape; * - \a functor is a numerical functor value * * The numerical functor is computed according to the shape's topological properties as follows: - * - orientation for vertices + * - orientation for vertices * - length for edges and wires * - area for faces and shells * - volume for solids, compounds, compsolids @@ -161,7 +161,7 @@ namespace GEOMUtils * \retval bool Returns false if the shape has no faces, i.e. impossible to build triangulation. */ Standard_EXPORT bool CheckTriangulation (const TopoDS_Shape& theShape); - + /*! * \brief Return type of shape for explode. In case of compound it will be a type of its first sub shape. * \param theShape The shape to get type of. @@ -201,7 +201,7 @@ namespace GEOMUtils Standard_EXPORT Standard_Real GetMinDistanceSingular(const TopoDS_Shape& aSh1, const TopoDS_Shape& aSh2, gp_Pnt& Ptmp1, gp_Pnt& Ptmp2); - + /*! * \brief Computes minumal distance between two shapes. * @@ -214,7 +214,7 @@ namespace GEOMUtils Standard_EXPORT Standard_Real GetMinDistance(const TopoDS_Shape& theShape1, const TopoDS_Shape& theShape2, gp_Pnt& thePnt1, gp_Pnt& thePnt2); - + /*! * \brief Computes normal projection of \a thePoint to \a theFace. * @@ -222,12 +222,14 @@ namespace GEOMUtils * \param theFace the face shape * \param theU the output U parameter of the point on the face * \param theV the output V parameter of the point on the face + * \param theTol the tolerance value. Maximum of theTol and 1e-04 will be used for calculation. * \retval the projection (3d point) if found, throws an exception otherwise */ Standard_EXPORT gp_Pnt ProjectPointOnFace(const gp_Pnt& thePoint, const TopoDS_Shape& theFace, - double& theU, double& theV); - + double& theU, double& theV, + const double theTol = 1e-04); + /*! * \brief Returns the point clicked in 3D view. * @@ -274,7 +276,7 @@ namespace GEOMUtils * operation or \c false otherwise */ Standard_EXPORT bool CheckBOPArguments(const TopoDS_Shape &theShape); - + /*! * \brief Limit shape tolerance to the given value * @@ -312,13 +314,13 @@ namespace GEOMUtils /*! * \brief Fix curves of the given shape - * + * * The function checks each curve of the input shape in the following way: * - compute deviation of the curve from the underlying surface in a set of points * computed with the certain discretization step value * - find maximum tolerance between computed deviation values * - limit tolerance of the curve with the computed maximum value - * + * * \param shape shape being fixed * \return \c true if resulting shape is valid */ @@ -332,7 +334,7 @@ namespace GEOMUtils */ Standard_EXPORT bool Write( const TopoDS_Shape& shape, const char* fileName ); - + /*! * \brief Extract single SOLID from COMPSOLID or COMPOUND. *