IPAL52698: Convert to quadratic is wrong (hard periodic surfaces)

This commit is contained in:
eap 2015-05-07 20:57:17 +03:00
parent 87ad2fe311
commit 062d52d5db
2 changed files with 138 additions and 66 deletions

View File

@ -813,18 +813,20 @@ namespace
}
//=======================================================================
//function : applyIn2D
//function : ApplyIn2D
//purpose : Perform given operation on two 2d points in parameric space of given surface.
// It takes into account period of the surface. Use gp_XY_FunPtr macro
// to easily define pointer to function of gp_XY class.
//=======================================================================
gp_XY SMESH_MesherHelper::applyIn2D(const Handle(Geom_Surface)& surface,
gp_XY SMESH_MesherHelper::ApplyIn2D(Handle(Geom_Surface) surface,
const gp_XY& uv1,
const gp_XY& uv2,
xyFunPtr fun,
const bool resultInPeriod)
{
if ( surface->IsKind(STANDARD_TYPE(Geom_RectangularTrimmedSurface )))
surface = Handle(Geom_RectangularTrimmedSurface)::DownCast( surface )->BasisSurface();
Standard_Boolean isUPeriodic = surface.IsNull() ? false : surface->IsUPeriodic();
Standard_Boolean isVPeriodic = surface.IsNull() ? false : surface->IsVPeriodic();
if ( !isUPeriodic && !isVPeriodic )
@ -852,6 +854,31 @@ gp_XY SMESH_MesherHelper::applyIn2D(const Handle(Geom_Surface)& surface,
return res;
}
//=======================================================================
//function : AdjustByPeriod
//purpose : Move node positions on a FACE within surface period
//=======================================================================
void SMESH_MesherHelper::AdjustByPeriod( const TopoDS_Face& face, gp_XY uv[], const int nbUV )
{
SMESH_MesherHelper h( *myMesh ), *ph = face.IsSame( myShape ) ? this : &h;
ph->SetSubShape( face );
for ( int iCoo = U_periodic; iCoo <= V_periodic; ++iCoo )
if ( ph->GetPeriodicIndex() & iCoo )
{
const double period = ( ph->myPar2[iCoo-1] - ph->myPar1[iCoo-1] );
const double xRef = uv[0].Coord( iCoo );
for ( int i = 1; i < nbUV; ++i )
{
double x = uv[i].Coord( iCoo );
double dx = ShapeAnalysis::AdjustByPeriod( x, xRef, period );
uv[i].SetCoord( iCoo, x + dx );
}
}
}
//=======================================================================
//function : GetMiddleUV
//purpose : Return middle UV taking in account surface period
@ -862,13 +889,13 @@ gp_XY SMESH_MesherHelper::GetMiddleUV(const Handle(Geom_Surface)& surface,
const gp_XY& p2)
{
// NOTE:
// the proper place of getting basic surface seems to be in applyIn2D()
// the proper place of getting basic surface seems to be in ApplyIn2D()
// but we put it here to decrease a risk of regressions just before releasing a version
Handle(Geom_Surface) surf = surface;
while ( !surf.IsNull() && surf->IsKind(STANDARD_TYPE(Geom_RectangularTrimmedSurface )))
surf = Handle(Geom_RectangularTrimmedSurface)::DownCast( surf )->BasisSurface();
// Handle(Geom_Surface) surf = surface;
// while ( !surf.IsNull() && surf->IsKind(STANDARD_TYPE(Geom_RectangularTrimmedSurface )))
// surf = Handle(Geom_RectangularTrimmedSurface)::DownCast( surf )->BasisSurface();
return applyIn2D( surf, p1, p2, & AverageUV );
return ApplyIn2D( surface, p1, p2, & AverageUV );
}
//=======================================================================
@ -1266,6 +1293,15 @@ const SMDS_MeshNode* SMESH_MesherHelper::GetCentralNode(const SMDS_MeshNode* n1,
if ( solidID < 1 && !faceId2nbNodes.empty() ) // SOLID not found
{
// find ID of the FACE the four corner nodes belong to
itMapWithIdFace = faceId2nbNodes.find( myShapeID ); // IPAL52698
if ( itMapWithIdFace != faceId2nbNodes.end() &&
itMapWithIdFace->second == 4 )
{
shapeType = TopAbs_FACE;
faceID = myShapeID;
}
else
{
itMapWithIdFace = faceId2nbNodes.begin();
for ( ; itMapWithIdFace != faceId2nbNodes.end(); ++itMapWithIdFace)
{
@ -1277,6 +1313,7 @@ const SMDS_MeshNode* SMESH_MesherHelper::GetCentralNode(const SMDS_MeshNode* n1,
}
}
}
}
TopoDS_Face F;
if ( shapeType == TopAbs_FACE )
@ -1291,11 +1328,20 @@ const SMDS_MeshNode* SMESH_MesherHelper::GetCentralNode(const SMDS_MeshNode* n1,
bool toCheck = true;
if ( !F.IsNull() && !force3d )
{
uvAvg = calcTFI (0.5, 0.5,
GetNodeUV(F,n1,n3,&toCheck), GetNodeUV(F,n2,n4,&toCheck),
GetNodeUV(F,n3,n1,&toCheck), GetNodeUV(F,n4,n2,&toCheck),
GetNodeUV(F,n12,n3), GetNodeUV(F,n23,n4),
GetNodeUV(F,n34,n2), GetNodeUV(F,n41,n2));
gp_XY uv[8] = {
GetNodeUV( F,n1, n3, &toCheck ),
GetNodeUV( F,n2, n4, &toCheck ),
GetNodeUV( F,n3, n1, &toCheck ),
GetNodeUV( F,n4, n2, &toCheck ),
GetNodeUV( F,n12, n3 ),
GetNodeUV( F,n23, n4 ),
GetNodeUV( F,n34, n2 ),
GetNodeUV( F,n41, n2 )
};
AdjustByPeriod( F, uv, 8 ); // put uv[] within a period (IPAL52698)
uvAvg = calcTFI (0.5, 0.5, uv[0],uv[1],uv[2],uv[3], uv[4],uv[5],uv[6],uv[7] );
TopLoc_Location loc;
Handle( Geom_Surface ) S = BRep_Tool::Surface( F, loc );
P = S->Value( uvAvg.X(), uvAvg.Y() ).Transformed( loc );
@ -1411,6 +1457,15 @@ const SMDS_MeshNode* SMESH_MesherHelper::GetCentralNode(const SMDS_MeshNode* n1,
if ( solidID < 1 && !faceId2nbNodes.empty() ) // SOLID not found
{
// find ID of the FACE the four corner nodes belong to
itMapWithIdFace = faceId2nbNodes.find( myShapeID ); // IPAL52698
if ( itMapWithIdFace != faceId2nbNodes.end() &&
itMapWithIdFace->second == 4 )
{
shapeType = TopAbs_FACE;
faceID = myShapeID;
}
else
{
itMapWithIdFace = faceId2nbNodes.begin();
for ( ; itMapWithIdFace != faceId2nbNodes.end(); ++itMapWithIdFace)
{
@ -1422,6 +1477,7 @@ const SMDS_MeshNode* SMESH_MesherHelper::GetCentralNode(const SMDS_MeshNode* n1,
}
}
}
}
TopoDS_Face F;
gp_XY uvAvg;
@ -1430,13 +1486,18 @@ const SMDS_MeshNode* SMESH_MesherHelper::GetCentralNode(const SMDS_MeshNode* n1,
{
F = TopoDS::Face( meshDS->IndexToShape( faceID ));
bool checkOK = true, badTria = false;
gp_XY uv1 = GetNodeUV( F, n1, n23, &checkOK );
gp_XY uv2 = GetNodeUV( F, n2, n31, &checkOK );
gp_XY uv3 = GetNodeUV( F, n3, n12, &checkOK );
gp_XY uv12 = GetNodeUV( F, n12, n3, &checkOK );
gp_XY uv23 = GetNodeUV( F, n23, n1, &checkOK );
gp_XY uv31 = GetNodeUV( F, n31, n2, &checkOK );
uvAvg = GetCenterUV( uv1,uv2,uv3, uv12,uv23,uv31, &badTria );
gp_XY uv[6] = {
GetNodeUV( F, n1, n23, &checkOK ),
GetNodeUV( F, n2, n31, &checkOK ),
GetNodeUV( F, n3, n12, &checkOK ),
GetNodeUV( F, n12, n3, &checkOK ),
GetNodeUV( F, n23, n1, &checkOK ),
GetNodeUV( F, n31, n2, &checkOK )
};
AdjustByPeriod( F, uv, 6 ); // put uv[] within a period (IPAL52698)
uvAvg = GetCenterUV( uv[0],uv[1],uv[2], uv[3],uv[4],uv[5], &badTria );
if ( badTria || !checkOK )
force3d = true;
}
@ -4875,7 +4936,7 @@ void SMESH_MesherHelper::FixQuadraticElements(SMESH_ComputeErrorPtr& compError,
gp_XY uv2 = faceHlp.GetNodeUV( face, link->node2(), nodeOnFace, &checkUV );
gp_XY uv12 = faceHlp.GetMiddleUV( surf, uv1, uv2 );
// uvMove = uvm - uv12
gp_XY uvMove = applyIn2D(surf, uvm, uv12, gp_XY_Subtracted, /*inPeriod=*/false);
gp_XY uvMove = ApplyIn2D(surf, uvm, uv12, gp_XY_Subtracted, /*inPeriod=*/false);
( is1 ? move1 : move0 ).SetCoord( uvMove.X(), uvMove.Y(), 0 );
if ( !is1 ) // correct nodeOnFace for move1 (issue 0020919)
nodeOnFace = (*(++chain.rbegin()))->_mediumNode;
@ -4942,7 +5003,7 @@ void SMESH_MesherHelper::FixQuadraticElements(SMESH_ComputeErrorPtr& compError,
// compute 3D displacement by 2D one
Handle(Geom_Surface) s = BRep_Tool::Surface(face,loc);
gp_XY oldUV = faceHlp.GetNodeUV( face, (*link1)->_mediumNode, 0, &checkUV );
gp_XY newUV = applyIn2D( s, oldUV, gp_XY( move.X(),move.Y()), gp_XY_Added );
gp_XY newUV = ApplyIn2D( s, oldUV, gp_XY( move.X(),move.Y()), gp_XY_Added );
gp_Pnt newPnt = s->Value( newUV.X(), newUV.Y());
move = gp_Vec( XYZ((*link1)->_mediumNode), newPnt.Transformed(loc) );
if ( SMDS_FacePosition* nPos =
@ -5051,6 +5112,7 @@ void SMESH_MesherHelper::FixQuadraticElements(SMESH_ComputeErrorPtr& compError,
if ( i > 3 && nodes[i]->GetPosition()->GetTypeOfPosition() == SMDS_TOP_FACE )
CheckNodeUV( F, nodes[i], uv[ i ], 2*tol, /*force=*/true );
}
AdjustByPeriod( F, uv, 8 ); // put uv[] within a period (IPAL52698)
// move the central node
gp_XY uvCent = calcTFI (0.5, 0.5, uv[0],uv[1],uv[2],uv[3],uv[4],uv[5],uv[6],uv[7] );
gp_Pnt p = surf->Value( uvCent.X(), uvCent.Y() ).Transformed( loc );
@ -5076,7 +5138,7 @@ void SMESH_MesherHelper::FixQuadraticElements(SMESH_ComputeErrorPtr& compError,
// nodes
nodes.assign( tria->begin_nodes(), tria->end_nodes() );
// UV
bool uvOK = true, badTria;
bool uvOK = true, badTria = false;
for ( int i = 0; i < 6; ++i )
{
uv[ i ] = GetNodeUV( F, nodes[i], nodes[(i+1)%3], &uvOK );
@ -5085,6 +5147,7 @@ void SMESH_MesherHelper::FixQuadraticElements(SMESH_ComputeErrorPtr& compError,
if ( nodes[i]->GetPosition()->GetTypeOfPosition() == SMDS_TOP_FACE )
CheckNodeUV( F, nodes[i], uv[ i ], 2*tol, /*force=*/true );
}
// move the central node
gp_Pnt p;
if ( !uvOK || badTria )
@ -5095,6 +5158,7 @@ void SMESH_MesherHelper::FixQuadraticElements(SMESH_ComputeErrorPtr& compError,
}
else
{
AdjustByPeriod( F, uv, 6 ); // put uv[] within a period (IPAL52698)
gp_XY uvCent = GetCenterUV( uv[0], uv[1], uv[2], uv[3], uv[4], uv[5], &badTria );
p = surf->Value( uvCent.X(), uvCent.Y() ).Transformed( loc );
}

View File

@ -482,10 +482,10 @@ public:
bool * isBadTria=0);
/*!
* \brief Define a pointer to wrapper over a function of gp_XY class,
* suitable to pass as xyFunPtr to applyIn2D().
* suitable to pass as xyFunPtr to ApplyIn2D().
* For exaple gp_XY_FunPtr(Added) defines pointer gp_XY_Added to function
* calling gp_XY::Added(gp_XY), which is to be used like following
* applyIn2D(surf, uv1, uv2, gp_XY_Added)
* ApplyIn2D(surf, uv1, uv2, gp_XY_Added)
*/
#define gp_XY_FunPtr(meth) \
static gp_XY __gpXY_##meth (const gp_XY& uv1, const gp_XY& uv2) { return uv1.meth( uv2 ); } \
@ -496,12 +496,20 @@ public:
* It takes into account period of the surface. Use gp_XY_FunPtr macro
* to easily define pointer to function of gp_XY class.
*/
static gp_XY applyIn2D(const Handle(Geom_Surface)& surface,
static gp_XY ApplyIn2D(Handle(Geom_Surface) surface,
const gp_XY& uv1,
const gp_XY& uv2,
xyFunPtr fun,
const bool resultInPeriod=true);
/*!
* \brief Move node positions on a FACE within surface period
* \param [in] face - the FACE
* \param [inout] uv - node positions to adjust
* \param [in] nbUV - nb of \a uv
*/
void AdjustByPeriod( const TopoDS_Face& face, gp_XY uv[], const int nbUV );
/*!
* \brief Check if inFaceNode argument is necessary for call GetNodeUV(F,..)
* \retval bool - return true if the face is periodic