View Issue Details

IDProjectCategoryView StatusLast Update
0028601Open CASCADEOCCT:Shape Healingpublic2017-09-29 16:25
ReportermsvAssigned Tobugmaster  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version7.1.0 
Target Version7.2.0Fixed in Version7.2.0 
Summary0028601: ShapeFix_Shape increases tolerance of sub-shapes of a valid shape
DescriptionEven for a valid shape like box the command fixshape increases tolerance of vertices and edges.
And if we repeat fixshape on the result it again increases tolerances.
Steps To Reproducebox b 10 10 10
tolerance b

# Initial tolerances:
#Tolerance MAX=9.9999999999999995e-008 AVG=9.9999999999999995e-008 MIN=9.9999999999999995e-008
#FACE : MAX=9.9999999999999995e-008 AVG=9.9999999999999995e-008 MIN=9.9999999999999995e-008
#EDGE : MAX=9.9999999999999995e-008 AVG=9.9999999999999995e-008 MIN=9.9999999999999995e-008
#VERTEX : MAX=9.9999999999999995e-008 AVG=9.9999999999999995e-008 MIN=9.9999999999999995e-008

fixshape b1 b
tolerance b1

# Result tolerances:
#Tolerance MAX=1.0001000025000001e-007 AVG=1.0000923099999988e-007 MIN=9.9999999999999995e-008
#FACE : MAX=9.9999999999999995e-008 AVG=9.9999999999999995e-008 MIN=9.9999999999999995e-008
#EDGE : MAX=1.0001000025000001e-007 AVG=1.0001000025000001e-007 MIN=1.0001000025000001e-007
#VERTEX : MAX=1.0001000025000001e-007 AVG=1.0001000025000001e-007 MIN=1.0001000025000001e-007

fixshape b2 b1
tolerance b2

# Next result tolerances:
Tolerance MAX=1.0002000150005004e-007 AVG=1.0001846292312312e-007 MIN=9.9999999999999995e-008
FACE : MAX=9.9999999999999995e-008 AVG=9.9999999999999995e-008 MIN=9.9999999999999995e-008
EDGE : MAX=1.0002000150005004e-007 AVG=1.0002000150005004e-007 MIN=1.0002000150005004e-007
VERTEX : MAX=1.0002000150005004e-007 AVG=1.0002000150005004e-007 MIN=1.0002000150005004e-007
TagsNo tags attached.
Test case numberbugs modalg_6 bug28601

Relationships

related to 0028602 assignedisn Prevent ShapeFix_Shape to modify the input shape 

Activities

git

2017-04-14 15:17

administrator   ~0065230

Branch CR28601 has been created by imn.

SHA-1: c45b9971d94cc97e4aa3003a801e0793c8e10978


Detailed log of new commits:

Author: imn
Date: Fri Apr 14 15:15:18 2017 +0300

    0028601: ShapeFix_Shape increases tolerance of sub-shapes of a valid shape
    - If tolerance hasn't been changed then 'fixshape' will not change it.

imn

2017-04-14 15:21

developer   ~0065231

Dear Mikhail, could you please review CR28601.

msv

2017-04-18 09:57

developer   ~0065279

Remarks:

- Change commit message:
Avoid increase of edge/vertex tolerance if the distance between 3D curve and p-curve is within old tolerance.

src\ShapeFix\ShapeFix.cxx
- Avoid unneeded computation (line 227) if isChanged is false.

tests\bugs\modalg_6\bug28601
- remove 'checkview' command, as the picture does not contain anything worth to check.
- remove tab characters in the file and align lines properly.

git

2017-04-18 11:41

administrator   ~0065282

Branch CR28601_1 has been created by imn.

SHA-1: 3629d44c0138bb1fe00f3e38562ae813d431c357


Detailed log of new commits:

Author: imn
Date: Fri Apr 14 15:15:18 2017 +0300

    0028601: ShapeFix_Shape increases tolerance of sub-shapes of a valid shape
    - Avoid increase of edge/vertex tolerance if the distance between 3D curve and p-curve is within old tolerance.

imn

2017-04-18 11:44

developer   ~0065283

Dear Mikhail, could you please review CR28601_1, branch is updated according to remarks

msv

2017-04-18 14:48

developer   ~0065295

Reviewed.

mkv

2017-04-19 19:37

tester   ~0065368

Dear BugMaster,
Branch CR28601_1 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode.
SHA-1: 3629d44c0138bb1fe00f3e38562ae813d431c357

Number of compiler warnings:

occt component :
Linux: 0 (0 on master)
Windows: 0 (0 on master)
MacOS : 0 (0 on master)

products component :
Linux: 64 (64 on master)
Windows: 0 (0 on master)
MacOS : 1209

Regressions/Differences/Improvements:
http://occt-tests/CR28601_1-master-OCCT/Debian70-64/bugs/heal/bug329.html
http://occt-tests/CR28601_1-master-OCCT/Windows-64-VC10/bugs/heal/bug329.html
Improvements:
  bugs heal bug329

Testing cases:
http://occt-tests/CR28601_1-master-OCCT/Debian70-64/bugs/modalg_6/bug28601.html
http://occt-tests/CR28601_1-master-OCCT/Windows-64-VC10/bugs/modalg_6/bug28601.html
bugs modalg_6 bug28601: OK

Testing on Linux:
occt component :
Total MEMORY difference: 91811326 / 90701406 [+1.22%]
Total CPU difference: 19810.100000000286 / 19728.26000000033 [+0.41%]
products component :
Total MEMORY difference: 30349096 / 30341918 [+0.02%]
Total CPU difference: 5370.089999999985 / 5356.299999999965 [+0.26%]
Testing on Windows:
occt component :
Total MEMORY difference: 57806030 / 57804189 [+0.00%]
Total CPU difference: 18296.09368189848 / 18136.473458698543 [+0.88%]
products component :
Total MEMORY difference: 22303813 / 22265369 [+0.17%]
Total CPU difference: 5365.669995099962 / 5359.788757399982 [+0.11%]

There are following differences in images found by testdiff.
http://occt-tests/CR28601_1-master-Products/Debian70-64/diff-Debian70-64.html
http://occt-tests/CR28601_1-master-Products/Windows-64-VC10/diff-Windows-64-VC10-image.html
IMAGE sat doc_6 H4: H4.png differs

git

2017-04-19 19:38

administrator   ~0065369

Branch CR28601_1 has been updated by mkv.

SHA-1: c7d56f462b5a8407dd37c6907f898087885f1bce


Detailed log of new commits:

Author: mkv
Date: Wed Apr 19 19:37:27 2017 +0300

    // Small correction of test case for issue CR28601

mkv

2017-04-19 19:38

tester   ~0065370

Dear BugMaster,
Test case bugs/heal/bug329 was corrected and pushed to branch CR28601_1, now it is OK.

mkv

2017-04-19 19:38

tester   ~0065371

Dear imn,
Branch CR28601_1 has been rejected due to:
- differences in images

imn

2017-04-20 16:07

developer   ~0065401

Differences in images are due to losses of curves during read/write source file. Now 4 more curves, which source shape contains, is founded and displayed, it's improvement.

msv

2017-04-20 16:54

developer   ~0065405

I agree, please test.

msv

2017-04-20 16:56

developer   ~0065406

Sorry, already tested, so integrate.

mkv

2017-04-21 17:20

tester   ~0065433

Dear BugMaster,
Branch CR28601_1 is TESTED.

git

2017-05-12 11:36

administrator   ~0065944

Branch CR28601 has been deleted by kgv.

SHA-1: c45b9971d94cc97e4aa3003a801e0793c8e10978

git

2017-05-12 11:36

administrator   ~0065945

Branch CR28601_1 has been deleted by kgv.

SHA-1: c7d56f462b5a8407dd37c6907f898087885f1bce

Related Changesets

occt: master 0af424c2

2017-04-14 12:15:18

imn


Committer: bugmaster Details Diff
0028601: ShapeFix_Shape increases tolerance of sub-shapes of a valid shape
- Avoid increase of edge/vertex tolerance if the distance between 3D curve and p-curve is within old tolerance.
Affected Issues
0028601
mod - src/ShapeFix/ShapeFix.cxx Diff File
mod - tests/bugs/heal/bug329 Diff File
add - tests/bugs/modalg_6/bug28601 Diff File

Issue History

Date Modified Username Field Change
2017-03-28 11:42 msv New Issue
2017-03-28 11:42 msv Assigned To => gka
2017-03-28 11:49 msv Relationship added related to 0028602
2017-04-04 16:19 msv Assigned To gka => imn
2017-04-04 16:19 msv Status new => assigned
2017-04-14 15:17 git Note Added: 0065230
2017-04-14 15:21 imn Note Added: 0065231
2017-04-14 15:21 imn Assigned To imn => msv
2017-04-14 15:21 imn Status assigned => resolved
2017-04-18 09:57 msv Note Added: 0065279
2017-04-18 09:57 msv Assigned To msv => imn
2017-04-18 09:57 msv Status resolved => assigned
2017-04-18 11:41 git Note Added: 0065282
2017-04-18 11:44 imn Note Added: 0065283
2017-04-18 11:44 imn Assigned To imn => msv
2017-04-18 11:44 imn Status assigned => resolved
2017-04-18 14:48 msv Note Added: 0065295
2017-04-18 14:48 msv Assigned To msv => bugmaster
2017-04-18 14:48 msv Status resolved => reviewed
2017-04-18 15:42 mkv Assigned To bugmaster => mkv
2017-04-19 19:37 mkv Note Added: 0065368
2017-04-19 19:38 git Note Added: 0065369
2017-04-19 19:38 mkv Note Added: 0065370
2017-04-19 19:38 mkv Note Added: 0065371
2017-04-19 19:38 mkv Assigned To mkv => imn
2017-04-19 19:38 mkv Status reviewed => feedback
2017-04-19 19:39 mkv Test case number => bugs modalg_6 bug28601
2017-04-20 16:07 imn Note Added: 0065401
2017-04-20 16:07 imn Assigned To imn => msv
2017-04-20 16:07 imn Status feedback => resolved
2017-04-20 16:07 imn Status resolved => feedback
2017-04-20 16:54 msv Note Added: 0065405
2017-04-20 16:55 msv Assigned To msv => mkv
2017-04-20 16:56 msv Note Added: 0065406
2017-04-21 17:20 mkv Note Added: 0065433
2017-04-21 17:20 mkv Assigned To mkv => bugmaster
2017-04-21 17:20 mkv Status feedback => tested
2017-04-28 13:08 bugmaster Changeset attached => occt master 0af424c2
2017-04-28 13:08 bugmaster Status tested => verified
2017-04-28 13:08 bugmaster Resolution open => fixed
2017-05-12 11:36 git Note Added: 0065944
2017-05-12 11:36 git Note Added: 0065945
2017-09-29 16:19 aiv Fixed in Version => 7.2.0
2017-09-29 16:25 aiv Status verified => closed