0029858: Modeling Data - Regression in GeomAPI_ExtremaCurveCurve

ID	Project	Category	View Status	Date Submitted	Last Update

0029858	Community	OCCT:Modeling Data	public	2018-06-11 16:41	2019-09-15 10:55

Reporter	martinsiggel	Assigned To	bugmaster
Priority	normal	Severity	major
Status	closed	Resolution	fixed
Product Version	7.2.0
Target Version	7.4.0	Fixed in Version	7.4.0

Summary	0029858: Modeling Data - Regression in GeomAPI_ExtremaCurveCurve
Description	Commit 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 Reproduce	test lowalgos extcc bug29858_01 test lowalgos extcc bug29858_02 test lowalgos extcc bug29858_03 test lowalgos extcc bug29858_04
Tags	No tags attached.

Test case number	lowalgos/extcc/bug29858_01, lowalgos/extcc/bug29858_02, lowalgos/extcc/bug29858_03, lowalgos/extcc/bug29858_04

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

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

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

View Issue Details

Attached Files

Activities

Related Changesets

Issue History