View Issue Details

IDProjectCategoryView StatusLast Update
0029858CommunityOCCT:Modeling Datapublic2019-09-15 10:55
Reportermartinsiggel Assigned Tobugmaster  
PrioritynormalSeveritymajor 
Status closedResolutionfixed 
Product Version7.2.0 
Target Version7.4.0Fixed in Version7.4.0 
Summary0029858: Modeling Data - Regression in GeomAPI_ExtremaCurveCurve
DescriptionCommit http://git.dev.opencascade.org/gitweb/?p=occt.git;a=commit;h=246c7a7554dd24047ada720fbfce42e9c10b4657 introduced a regression, computing the wrong intersection / extrema results.

The regression either computes the wrong result or the wrong number of extremas.

Steps To Reproducetest lowalgos extcc bug29858_01
test lowalgos extcc bug29858_02
test lowalgos extcc bug29858_03
test lowalgos extcc bug29858_04
TagsNo tags attached.
Test case numberlowalgos/extcc/bug29858_01, lowalgos/extcc/bug29858_02, lowalgos/extcc/bug29858_03, lowalgos/extcc/bug29858_04

Attached Files

  • geomapi_extremacurvecurve_regression.tar.gz (17,545 bytes)
  • shapes.zip (18,815 bytes)

Activities

martinsiggel

2018-06-11 16:41

developer  

geomapi_extremacurvecurve_regression.tar.gz (17,545 bytes)

git

2019-09-04 13:01

administrator   ~0086730

Branch CR29858 has been created by aml.

SHA-1: afcaf36de312fbae3b6f0de7cfe0e0bdcbc7257f


Detailed log of new commits:

Author: aml
Date: Tue Sep 3 16:08:54 2019 +0300

    0029858: Modeling Data - Regression in GeomAPI_ExtremaCurveCurve
    
    Fix Newton's minimum projection algorithm onto boundaries in case of conditional optimization.
    Add possibility to detect several optimal points.

git

2019-09-05 14:27

administrator   ~0086795

Branch CR29858 has been updated forcibly by aml.

SHA-1: f59bd7dcc0c196f7c40cbbe89e3592811ff6e5c6

git

2019-09-06 08:15

administrator   ~0086831

Branch CR29858 has been updated forcibly by aml.

SHA-1: c3b519a7fcc101c52ca899c7c1148f2bcc48e0c8

aml

2019-09-06 14:38

developer  

shapes.zip (18,815 bytes)

aml

2019-09-06 14:40

developer   ~0086884

Dear Mikhail,
Could you please add shapes for Jenkins testing? They are located in the "shapes.zip" archive.

msv

2019-09-06 15:04

developer   ~0086887

Done.

git

2019-09-09 06:41

administrator   ~0086974

Branch CR29858 has been updated forcibly by aml.

SHA-1: ec4d52e073674cb73c6923a4f32e6204c48bf7a8

aml

2019-09-09 08:42

developer   ~0086975

Dear Mikhail,

Could you please review the CR29858 branch?

OCCT testing:
http://occt-tests/CR29858-master-aml-OCCT/Windows-64-VC14/diff_summary.html
http://occt-tests/CR29858-master-aml-OCCT/Debian80-64/diff_summary.html

Products testing:
http://occt-tests/CR29858-master-aml-Products/Windows-64-VC14/diff_summary.html
http://occt-tests/CR29858-master-aml-Products/Debian80-64/diff-Debian80-64-cpu.html

msv

2019-09-09 12:05

developer   ~0086978

src/math/math_BFGS.cxx
- Please check if the sign of aDir is correctly treated in the changed condition (may be to reverse?).

Re-test with new reference.

git

2019-09-09 12:32

administrator   ~0086979

Branch CR29858 has been updated by aml.

SHA-1: 06ba26ca4234b6a1bc8e138aa793f70192e9211f


Detailed log of new commits:

Author: aml
Date: Mon Sep 9 12:28:53 2019 +0300

    # Remark correction.

aml

2019-09-09 14:33

developer   ~0086980

Dear Mikhail,

Your remark was applied. Could you please check the CR29858 branch again?

OCCT testing:
http://occt-tests/CR29858-master-aml-OCCT/Windows-64-VC14/diff_summary.html
http://occt-tests/CR29858-master-aml-OCCT/Debian80-64/diff_summary.html

Products testing:
http://occt-tests/CR29858-master-aml-Products/Windows-64-VC14/diff_summary.html
http://occt-tests/CR29858-master-aml-Products/Debian80-64/diff_summary.html

msv

2019-09-09 15:20

developer   ~0086984

Reviewed.

bugmaster

2019-09-09 18:54

administrator   ~0086988

Combination -
OCCT branch : CR29858
master SHA - 06ba26ca4234b6a1bc8e138aa793f70192e9211f
5f5b1aed1c6e139bbd34314eca77ae7abcd8895c
Products branch : master SHA - f9d0bd5e3a29d6a97b3f5f6354ea2397253ab4f8
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: 16818.880000000147 / 16836.290000000165 [-0.10%]
Products
Total CPU difference: 10566.830000000013 / 10553.510000000031 [+0.13%]
Windows-64-VC14:
OCCT
Total CPU difference: 18312.875 / 18300.0625 [+0.07%]
Products
Total CPU difference: 12372.703125 / 12227.453125 [+1.19%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention

git

2019-09-15 10:55

administrator   ~0087100

Branch CR29858 has been deleted by inv.

SHA-1: 06ba26ca4234b6a1bc8e138aa793f70192e9211f

Related Changesets

occt: master 94beb42a

2019-09-03 13:08:54

aml


Committer: bugmaster Details Diff
0029858: Modeling Data - Regression in GeomAPI_ExtremaCurveCurve

Fix Newton's minimum projection algorithm onto boundaries in case of conditional optimization.
Add possibility to detect several optimal points at initialization of the math_GlobOptMin.
Affected Issues
0029858
mod - src/ChFi3d/ChFi3d_Builder_CnCrn.cxx Diff File
mod - src/math/math_BFGS.cxx Diff File
mod - src/math/math_GlobOptMin.cxx Diff File
mod - src/math/math_GlobOptMin.hxx Diff File
mod - src/math/math_NewtonMinimum.cxx Diff File
add - tests/lowalgos/extcc/bug29858_01 Diff File
add - tests/lowalgos/extcc/bug29858_02 Diff File
add - tests/lowalgos/extcc/bug29858_03 Diff File
add - tests/lowalgos/extcc/bug29858_04 Diff File

Issue History

Date Modified Username Field Change
2018-06-11 16:41 martinsiggel New Issue
2018-06-11 16:41 martinsiggel Assigned To => msv
2018-06-11 16:41 martinsiggel File Added: geomapi_extremacurvecurve_regression.tar.gz
2018-06-11 16:47 martinsiggel Steps to Reproduce Updated
2019-08-12 17:45 msv Target Version 7.4.0 => 7.5.0
2019-09-03 13:15 aml Assigned To msv => aml
2019-09-04 11:55 kgv Summary Regression in GeomAPI_ExtremaCurveCurve => Modeling Data - Regression in GeomAPI_ExtremaCurveCurve
2019-09-04 13:01 git Note Added: 0086730
2019-09-04 13:01 aml Status new => assigned
2019-09-05 14:27 git Note Added: 0086795
2019-09-06 08:15 git Note Added: 0086831
2019-09-06 14:38 aml File Added: shapes.zip
2019-09-06 14:40 aml Note Added: 0086884
2019-09-06 14:40 aml Assigned To aml => msv
2019-09-06 14:40 aml Status assigned => feedback
2019-09-06 15:04 msv Note Added: 0086887
2019-09-06 15:04 msv Assigned To msv => aml
2019-09-06 15:04 msv Status feedback => assigned
2019-09-06 16:56 msv Target Version 7.5.0 => 7.4.0
2019-09-09 06:41 git Note Added: 0086974
2019-09-09 08:42 aml Note Added: 0086975
2019-09-09 08:42 aml Assigned To aml => msv
2019-09-09 08:42 aml Status assigned => resolved
2019-09-09 08:42 aml Steps to Reproduce Updated
2019-09-09 12:05 msv Note Added: 0086978
2019-09-09 12:05 msv Assigned To msv => aml
2019-09-09 12:05 msv Status resolved => assigned
2019-09-09 12:32 git Note Added: 0086979
2019-09-09 14:33 aml Note Added: 0086980
2019-09-09 14:33 aml Assigned To aml => msv
2019-09-09 14:33 aml Status assigned => resolved
2019-09-09 15:20 msv Note Added: 0086984
2019-09-09 15:20 msv Assigned To msv => bugmaster
2019-09-09 15:20 msv Status resolved => reviewed
2019-09-09 18:52 bugmaster Test case number => lowalgos/extcc/bug29858_01, lowalgos/extcc/bug29858_02, lowalgos/extcc/bug29858_03, lowalgos/extcc/bug29858_04
2019-09-09 18:54 bugmaster Note Added: 0086988
2019-09-09 18:54 bugmaster Status reviewed => tested
2019-09-15 10:51 bugmaster Changeset attached => occt master 94beb42a
2019-09-15 10:51 bugmaster Status tested => verified
2019-09-15 10:51 bugmaster Resolution open => fixed
2019-09-15 10:55 git Note Added: 0087100