MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0029698Open CASCADE[OCCT] OCCT:Modeling Algorithmspublic2018-04-16 15:042018-04-19 10:09
Reporteremv 
Assigned Tobugmaster 
PrioritynormalSeverityminor 
StatusreviewedResolutionopen 
PlatformOSOS Version
Product Version[OCCT] 7.2.0 
Target Version[OCCT] 7.3.0Fixed in Version 
Summary0029698: Regression vs 7.2.0: Common operation raises FLT_INVALID_OPERATION exception
DescriptionThe problem is reproduced in the test case bugs/modalg_6/bug26952_1 if "USE_FPE_SIGNAL" turned on.

The problem seems to be caused by the fix for the issue #29688.
Steps To Reproducetest bugs modalg_6 bug26952_1
TagsNo tags attached.
Test case number
Attached Files

- Relationships

-  Notes
(0075464)
git (administrator)
2018-04-16 15:21

Branch CR29698 has been created by emv.

SHA-1: e52d773ad46c6fbd56f9bdbd1972c0062abddeee


Detailed log of new commits:

Author: emv
Date: Mon Apr 16 15:21:35 2018 +0300

    0029698: Regression vs 7.2.0: Common operation raises FLT_INVALID_OPERATION exception
    
    When checking if the split edge is oriented as the original one (BOPTools_AlgoTools::IsSplitToReverse()) the tangent vectors should be computed for both edges at the same point. This point is taken on the split edge and projected on the original edge. The fix is intended to ensuring that the reference point will be taken inside the valid range of the split edge (i.e. not covered by the tolerance spheres of its bounding vertices) and the projection of this point on the original edge will be successful.
(0075468)
emv (developer)
2018-04-17 08:04

Dear Mikhail, could you please review the git branch CR29698?
Test results - http://jenkins-test-11.nnov.opencascade.com:8080/view/CR29698-master-emv/view/COMPARE/ [^]
(0075472)
msv (developer)
2018-04-17 12:24

src/BOPAlgo/BOPAlgo_Builder_2.cxx
- Put 828,831 lines in else statement.

src/BOPTools/BOPTools_AlgoTools2D.hxx
- Do not change API of AttachExistingPCurve. Instead, change IsSplitToReverse.

src/BOPTools/BOPTools_AlgoTools.cxx
- Make several intermediate points in IsSplitToReverse to increase reliability of the method. Add error status as the optional output parameter, and generate warning in the calling method in case of error.

Add a test case with "dsetsignal 1".
(0075477)
git (administrator)
2018-04-17 15:25

Branch CR29698_1 has been created by emv.

SHA-1: 9a58f7c24177a5653a1de1c7a6f3dfd1289fab89


Detailed log of new commits:

Author: emv
Date: Mon Apr 16 15:21:35 2018 +0300

    0029698: Regression vs 7.2.0: Common operation raises FLT_INVALID_OPERATION exception
    
    When checking if the split edge is oriented as the original one (BOPTools_AlgoTools::IsSplitToReverse()) the tangent vectors should be computed for both edges at the same point. This point is taken on the split edge and projected on the original edge.
    The fix is intended to ensuring that the reference point will be taken inside the valid range of the split edge (i.e. not covered by the tolerance spheres of its bounding vertices) and the projection of this point on the original edge will be successful. Moreover, several sampling points are now taken on the split edge and processed until first valid point is found.
    
    If requested (by a not null pointer) all *BOPTools_AlgoTools::IsSplitToReverse()* methods are now return the error status of the check. Before using the returned flag, the calling program should check this error status. For successful check the error status should be equal to zero.
    
    New warning *BOPAlgo_AlertUnableToOrientTheShape* is now returned in the algorithms in Boolean component in case the check for correct shape orientation failed.
    
    Test case for the issue.
(0075501)
emv (developer)
2018-04-18 08:18

Remarks have been considered. Please review the git branch CR29698_1.
(0075506)
msv (developer)
2018-04-18 10:50

In IsSplitToReverse(edge,edge) revise the error codes numbers.

Change theContext to const& in all APIs.
(0075522)
git (administrator)
2018-04-18 16:40

Branch CR29698_1 has been updated by emv.

SHA-1: ba9083f0faab10e5b02a3a0e94ab3c9bd09e1472


Detailed log of new commits:

Author: emv
Date: Wed Apr 18 16:40:38 2018 +0300

    # Considering remarks.

(0075532)
emv (developer)
2018-04-19 08:15

Please review again.
(0075535)
msv (developer)
2018-04-19 10:09

Reviewed.

- Issue History
Date Modified Username Field Change
2018-04-16 15:04 emv New Issue
2018-04-16 15:04 emv Assigned To => msv
2018-04-16 15:05 emv Assigned To msv => emv
2018-04-16 15:05 emv Status new => assigned
2018-04-16 15:21 git Note Added: 0075464
2018-04-17 08:04 emv Note Added: 0075468
2018-04-17 08:04 emv Assigned To emv => msv
2018-04-17 08:04 emv Status assigned => resolved
2018-04-17 12:24 msv Note Added: 0075472
2018-04-17 12:24 msv Assigned To msv => emv
2018-04-17 12:24 msv Status resolved => assigned
2018-04-17 15:25 git Note Added: 0075477
2018-04-18 08:18 emv Note Added: 0075501
2018-04-18 08:18 emv Assigned To emv => msv
2018-04-18 08:18 emv Status assigned => resolved
2018-04-18 10:50 msv Note Added: 0075506
2018-04-18 10:50 msv Assigned To msv => emv
2018-04-18 10:50 msv Status resolved => assigned
2018-04-18 16:40 git Note Added: 0075522
2018-04-19 08:15 emv Note Added: 0075532
2018-04-19 08:15 emv Assigned To emv => msv
2018-04-19 08:15 emv Status assigned => resolved
2018-04-19 10:09 msv Note Added: 0075535
2018-04-19 10:09 msv Assigned To msv => bugmaster
2018-04-19 10:09 msv Status resolved => reviewed


Copyright © 2000 - 2018 MantisBT Team
Powered by Mantis Bugtracker