OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-03 17:56:21 +03:00
Code Packages Releases Activity

0027087: [Regression to OCCT 6.7.1] BRepExtrema_DistShapeShape gives wrong result

Browse Source 
Extrema_ExtCs.cxx: number of subdivision is incresed for curve;

Modifications to fix regressions of some tests
IntTools_BeanFaceIntersector.cxx: restriction of curve, which is used in Extrema_ExtCS
ProjLib_CompProjectedCurve.cxx: tolerance is reduced for Extrema_ExtCS
BRepOffset_Tool.cxx: checking common vertex if it is Null

tests/bugs/modalg_7/bug27087 : test case added.

tests/bugs/modalg_5/bug25232_8 : test is modified according to new behavior of algorithms
This commit is contained in:
ifv 2019-09-03 15:48:24 +03:00 committed by bugmaster
parent 72e6020d50
commit 79aa9b5cf6
6 changed files with 36 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/BRepOffset/BRepOffset_Tool.cxx
View File

@ -1345,7 +1345,11 @@ static TopoDS_Edge AssembleEdge(const BOPDS_PDS& pDS,
{
TopoDS_Vertex CV, V11, V12, V21, V22;
TopExp::CommonVertex( CurEdge, anEdge, CV );
Standard_Boolean IsAutonomCV = IsAutonomVertex( CV, pDS, F1, F2 );
Standard_Boolean IsAutonomCV = Standard_False;
if (!CV.IsNull())
{
IsAutonomCV = IsAutonomVertex(CV, pDS, F1, F2);
}
if (IsAutonomCV)
{
aGlueTol = BRep_Tool::Tolerance(CV);

2
src/Extrema/Extrema_ExtCS.cxx
View File

@ -105,7 +105,7 @@ void Extrema_ExtCS::Perform(const Adaptor3d_Curve& C,
mySqDist.Clear();
Standard_Integer i, j;
Standard_Integer NbT, NbU, NbV;
NbT = NbU = NbV = 10;
NbT = 12; NbU = NbV = 10;
GeomAbs_CurveType myCtype = C.GetType();
myDone = Standard_False;

12
src/IntTools/IntTools_BeanFaceIntersector.cxx
View File

@ -913,7 +913,17 @@ void IntTools_BeanFaceIntersector::ComputeUsingExtremum()
}
GeomAdaptor_Curve aGACurve(aCurve, anarg1, anarg2);
Extrema_ExtCS theExtCS(aGACurve, aGASurface, Tol, Tol);
Extrema_ExtCS theExtCS;
theExtCS.Initialize(aGASurface, myUMinParameter, myUMaxParameter,
myVMinParameter, myVMaxParameter, Tol, Tol);
Standard_Real first = aCurve->FirstParameter(), last = aCurve->LastParameter();
if (aCurve->IsPeriodic() ||
(anarg1 >= first - Precision::PConfusion() && anarg2 <= last + Precision::PConfusion()))
{
//Extrema_ExtCS theExtCS(aGACurve, aGASurface, Tol, Tol);
theExtCS.Perform(aGACurve, anarg1, anarg2);
}
myExtrema = theExtCS;
if(myExtrema.IsDone() && (myExtrema.NbExt() || myExtrema.IsParallel())) {

4
src/ProjLib/ProjLib_CompProjectedCurve.cxx
View File

@ -636,8 +636,8 @@ void ProjLib_CompProjectedCurve::Init()
Standard_Boolean FromLastU = Standard_False,
isSplitsComputed = Standard_False;
const Standard_Real aTol3D = Precision::Confusion();
Extrema_ExtCS CExt(myCurve->Curve(), mySurface->Surface(), aTol3D, aTol3D);
const Standard_Real aTolExt = Precision::PConfusion();
Extrema_ExtCS CExt(myCurve->Curve(), mySurface->Surface(), aTolExt, aTolExt);
if (CExt.IsDone() && CExt.NbExt())
{
// Search for the minimum solution.

2
tests/bugs/modalg_5/bug25232_8
View File

@ -27,5 +27,5 @@ mkvolume result fcon3 fp
checkprops result -s 1706.51
checkshape result
checknbshapes result -vertex 5 -edge 6 -wire 2 -face 2 -shell 1 -solid 1 -compsolid 0 -compound 0 -shape 17
checknbshapes result -vertex 4 -edge 5 -wire 2 -face 2 -shell 1 -solid 1 -compsolid 0 -compound 0 -shape 15
checkview -display result -2d -path ${imagedir}/${test_image}.png

16
tests/bugs/modalg_7/bug27087 Normal file
View File

@ -0,0 +1,16 @@
puts "========"
puts "OCC27087"
puts "========"
puts ""
#######################################################
# [Regression to OCCT 6.7.1] BRepExtrema_DistShapeShape gives wrong result
#######################################################
smallview
restore [locate_data_file bug27087.brep] a
explode a
distmini r a_1 a_2
if { [dval r_val] > 0.0001 } { puts "Error: too big deviation, should be 0!" }
fit
checkview -screenshot -2d -path ${imagedir}/${test_image}.png