MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Related Changesets ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0026269Open CASCADE[OCCT] OCCT:Modeling Algorithmspublic2015-05-26 13:532021-09-04 16:22
Reporteraml 
Assigned Tobugmaster 
PrioritylowSeveritytweak 
StatusverifiedResolutionfixed 
PlatformOSOS Version
Product Version[OCCT] 6.9.0 
Target Version[OCCT] 7.6.0*Fixed in Version 
Summary0026269: Modeling Data - Analytical extrema does not take into account trimmed input data
DescriptionThere is problem with extrema package and its API:


Analytical extrema does not take into account trimmed input data. (case 1, case 2)

Steps To ReproduceTest cases
lowalgos extcc bug26269_1, bug26269_2

Reporter's scripts:
#case 1
line l1 0 0 0 1 0 0
line l2 2 0 0 1 0 0
trim l1 l1 0 1
trim l2 l2 0 1
extrema l1 l2

#case 2
circle c1 0 0 0 10
circle c2 0 0 0 5
trim c1 c1 0 1
trim c2 c2 3 4
extrema c1 c2
TagsNo tags attached.
Test case numbertests/lowalgos/extcc/bug26269_1,bug26269_2,bug29712_13,bug29712_21,bug29712_22,bug29712_24,bug29712_25,bug29712_29,bug29712_3,bug29712_32,bug29712_37,bug29712_5
Attached Files

- Relationships
related to 0026184closedbugmaster Open CASCADE GeomAPI_ExtremaCurveCurve hangs on parallel b-spline curves 
related to 0026063closedbugmaster Open CASCADE GeomAPI_ExtremaCurveSurface have inexact API 
related to 0026075closedbugmaster Open CASCADE Make Extrema_GenExtCC return IsParallel flag in case of parallel curves 
parent of 0027162closedabv Open CASCADE Draw command "(2d)extrema" incorrectly represent underlying algorithm results 
related to 0022560feedbackszy Community Extrema redesign 
related to 0022525resolvedszy Open CASCADE Improvement of Extrema 

-  Notes
(0042531)
aml (developer)
2015-06-29 12:11
edited on: 2015-06-29 12:12

ExtremaPC:

I
1) bad format of output:

proj c 3.0 3.0 2.0
 parameter 1 = 1.1738953633378706
ext_1 parameter 2 = 2.1611867552406454
ext_2 parameter 3 = 3.5000000059308389
ext_3 parameter 4 = 5.0399884503606023
ext_4 parameter 5 = 5.4804063007678074
ext_5 parameter 6 = 6.2485611263687888
ext_6 parameter 7 = 6.7381612209715556
ext_7 parameter 8 = 1.1738953633378706
ext_8

2) French comments:
proj c 3.999999652077201 5.0000000062915735 5.00002142991819367
 parameter 1 = 0.99910795389207607
ext_1 Le point est sur la courbe.
 Son parametre est U = 0.99910795389207607

II
There is no duplicates handling.

(0042533)
msv (developer)
2015-06-29 12:18

Andrey, please validate this bug fix.
(0042535)
abv (manager)
2015-06-29 12:40

I deem the proposed changes are quite reasonable, let's proceed (with low priority)
(0102308)
ifv (developer)
2021-07-06 17:18

Analysis
(0102362)
ifv (developer)
2021-07-08 11:45

Debugging
(0103021)
ifv (developer)
2021-08-06 10:16

Solution implementation
(0103027)
git (administrator)
2021-08-06 14:49

Branch CR26269 has been created by ifv.

SHA-1: 49413466cd154d7a642c544e36428d2df08786b5


Detailed log of new commits:

Author: ifv
Date: Fri Aug 6 14:49:14 2021 +0300

    0026269: Modeling Data - Analytical extrema does not take into account trimmed input data
    
    Treatment of trimmed data is added
    Extrema/Extrema_ExtCC.cxx
    
    Test cases added
    tests/lowalgos/extcc/bug26269_1
    tests/lowalgos/extcc/bug26269_2
(0103112)
git (administrator)
2021-08-10 12:42

Branch CR26269 has been updated forcibly by ifv.

SHA-1: 78fcd988a20bc62058d908483cc7cf6b3a361c28
(0103173)
git (administrator)
2021-08-13 11:24

Branch CR26269 has been updated forcibly by ifv.

SHA-1: 05428e70d22712faf667f7431a7ced3a204324e4
(0103226)
git (administrator)
2021-08-16 10:23

Branch CR26269 has been updated forcibly by ifv.

SHA-1: 4db8b2732920013437a6cee2ff142e4935e4ab0b
(0103230)
ifv (developer)
2021-08-16 11:57

Debugging
(0103237)
ifv (developer)
2021-08-16 12:55

Branch CR26269 is ready for review
Branches for integratin:
OCCT - CR26269
Products - not
(0103343)
jgv (developer)
2021-08-23 10:19

Remarks only for improvement of style and easiness of reading:

 - rename P1f,P1l,P2f,P2l to myP1f,myP1l,myP2f,myP2l;
 - rename myIsPar to myIsParallel

in the files Extrema_ExtCC.cxx, .hxx.
(0103608)
git (administrator)
2021-09-01 08:22

Branch CR26269 has been updated forcibly by jgv.

SHA-1: 69ccb38f2adc3150ff7b9420ab86351ba42d7f49
(0103614)
jgv (developer)
2021-09-01 12:11

Reviewed.

Branches for integration:
OCCT - CR26269
Products - not
(0103691)
jgv (developer)
2021-09-02 18:34

Reviewing
(0103756)
smoskvin (administrator)
2021-09-04 14:25

Combination -
OCCT branch : IR-2021-09-03
master SHA - f26ee38f2a309ffbf7de4eebbcef2c5a5c57d84e
a87b7ddc8cb44606b91e3f37113847c3f5f50fdc
Products branch : IR-2021-09-03 SHA - 87cca1a8f3dd94387a936b9d49f5bd719c69cf4d
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.

Number of compiler warnings:
No new/fixed warnings

Regressions/Differences/Improvements:
No regressions/differences

CPU differences:
Debian80-64:
OCCT
Total CPU difference: 17495.94000000042 / 17441.55000000031 [+0.31%]
Products
Total CPU difference: 11565.440000000113 / 11534.270000000102 [+0.27%]
Windows-64-VC14:
OCCT
Total CPU difference: 19307.140625 / 19200.421875 [+0.56%]
Products
Total CPU difference: 12917.859375 / 12874.53125 [+0.34%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
(0103779)
git (administrator)
2021-09-04 14:59

Branch CR26269 has been deleted by mnt.

SHA-1: 69ccb38f2adc3150ff7b9420ab86351ba42d7f49

- Related Changesets
occt: master 4391df89
Timestamp: 2021-08-06 11:49:14
Author: ifv
Committer: bugmaster
Details ] Diff ]
0026269: Modeling Data - Analytical extrema does not take into account trimmed input data

Treatment of trimmed data is added
Extrema/Extrema_ExtCC.cxx

Test cases added
tests/lowalgos/extcc/bug26269_1
tests/lowalgos/extcc/bug26269_2

Test cases are modified according to new behavior of algorithm
mod - src/Extrema/Extrema_ExtCC.cxx Diff ] File ]
mod - src/Extrema/Extrema_ExtCC.hxx Diff ] File ]
add - tests/lowalgos/extcc/bug26269_1 Diff ] File ]
add - tests/lowalgos/extcc/bug26269_2 Diff ] File ]
mod - tests/lowalgos/extcc/bug29712_13 Diff ] File ]
mod - tests/lowalgos/extcc/bug29712_21 Diff ] File ]
mod - tests/lowalgos/extcc/bug29712_22 Diff ] File ]
mod - tests/lowalgos/extcc/bug29712_24 Diff ] File ]
mod - tests/lowalgos/extcc/bug29712_25 Diff ] File ]
mod - tests/lowalgos/extcc/bug29712_29 Diff ] File ]
mod - tests/lowalgos/extcc/bug29712_3 Diff ] File ]
mod - tests/lowalgos/extcc/bug29712_32 Diff ] File ]
mod - tests/lowalgos/extcc/bug29712_37 Diff ] File ]
mod - tests/lowalgos/extcc/bug29712_5 Diff ] File ]

- Issue History
Date Modified Username Field Change
2015-05-26 13:53 aml New Issue
2015-05-26 13:53 aml Assigned To => msv
2015-05-26 13:54 aml Relationship added related to 0026184
2015-05-26 13:54 aml Relationship added related to 0026063
2015-05-26 13:55 aml Relationship added related to 0026075
2015-05-26 13:56 aml Description Updated View Revisions
2015-05-26 15:02 msv Description Updated View Revisions
2015-06-29 12:09 aml Relationship added related to 0026351
2015-06-29 12:11 aml Note Added: 0042531
2015-06-29 12:12 aml Note Edited: 0042531 View Revisions
2015-06-29 12:18 msv Note Added: 0042533
2015-06-29 12:18 msv Assigned To msv => abv
2015-06-29 12:18 msv Severity minor => tweak
2015-06-29 12:40 abv Note Added: 0042535
2015-06-29 12:40 abv Assigned To abv => aml
2015-06-29 12:40 abv Priority normal => low
2015-06-29 12:40 abv Status new => assigned
2016-02-16 14:02 aml Relationship added parent of 0027162
2016-07-07 07:04 aml Relationship added related to 0022560
2016-07-07 07:04 aml Relationship added related to 0022525
2016-10-28 16:32 msv Target Version 7.1.0 => 7.2.0
2017-07-24 09:25 msv Target Version 7.2.0 => Unscheduled
2021-05-14 17:34 szy Assigned To aml => ifv
2021-07-06 17:18 ifv Note Added: 0102308
2021-07-08 11:45 ifv Note Added: 0102362
2021-08-06 10:16 ifv Note Added: 0103021
2021-08-06 14:13 ifv Summary Extrema package revision => Modeling Data - Extrema package revision
2021-08-06 14:19 ifv Summary Modeling Data - Extrema package revision => Modeling Data - Analytical extrema does not take into account trimmed input data
2021-08-06 14:19 ifv Description Updated View Revisions
2021-08-06 14:49 git Note Added: 0103027
2021-08-10 12:42 git Note Added: 0103112
2021-08-13 11:24 git Note Added: 0103173
2021-08-16 10:23 git Note Added: 0103226
2021-08-16 11:57 ifv Note Added: 0103230
2021-08-16 12:55 ifv Note Added: 0103237
2021-08-16 12:55 ifv Assigned To ifv => jgv
2021-08-16 12:55 ifv Status assigned => resolved
2021-08-16 12:55 ifv Steps to Reproduce Updated View Revisions
2021-08-23 10:19 jgv Note Added: 0103343
2021-08-23 10:19 jgv Assigned To jgv => ifv
2021-08-23 10:19 jgv Status resolved => assigned
2021-09-01 08:22 git Note Added: 0103608
2021-09-01 12:10 jgv Assigned To ifv => jgv
2021-09-01 12:10 jgv Status assigned => resolved
2021-09-01 12:11 jgv Note Added: 0103614
2021-09-01 12:11 jgv Assigned To jgv => bugmaster
2021-09-01 12:11 jgv Status resolved => reviewed
2021-09-01 12:11 jgv Product Version 6.9.0 => 7.6.0*
2021-09-01 12:24 msv Product Version 7.6.0* => 6.9.0
2021-09-01 12:24 msv Target Version Unscheduled => 7.6.0*
2021-09-02 18:34 jgv Note Added: 0103691
2021-09-04 14:25 smoskvin Note Added: 0103756
2021-09-04 14:25 smoskvin Status reviewed => tested
2021-09-04 14:36 bugmaster Changeset attached => occt master 4391df89
2021-09-04 14:36 bugmaster Status tested => verified
2021-09-04 14:36 bugmaster Resolution open => fixed
2021-09-04 14:59 git Note Added: 0103779
2021-09-04 16:22 smoskvin Test case number => tests/lowalgos/extcc/bug26269_1,bug26269_2,bug29712_13,bug29712_21,bug29712_22,bug29712_24,bug29712_25,bug29712_29,bug29712_3,bug29712_32,bug29712_37,bug29712_5


Copyright © 2000 - 2021 MantisBT Team
Powered by Mantis Bugtracker