MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0028694Open CASCADE[OCCT] OCCT:Data Exchangepublic2017-04-26 18:102018-08-15 18:57
Reportermsv 
Assigned Toskl 
PrioritynormalSeverityminor 
StatusassignedResolutionopen 
PlatformOSOS Version
Product Version[OCCT] 7.1.0 
Target Version[OCCT] 7.4.0*Fixed in Version 
Summary0028694: IGES reader produces too small edge covered by its vertices
DescriptionThis bug has been discovered during fixing #23731 as a regression on "test bugs heal bug26671". Actually the shape got by reading from IGES file contains invalidity that is not reported by the current version of checkshape.
It is needed to analyze this case and understand whether the bug is in the IGES file or in the reader algorithm.
Steps To Reproduceigesbrep [locate_data_file bug26671.igs] r *

# check that all edges have enough length
foreach e [explode r e] {
  if {[lsearch [dump $e] degenerated] == -1 &&
      [validrange $e] == "edge has no valid range"} {
    regexp {Mass : *([0-9\-+.eE]+)} [lprops $e] dummy len
    regexp {Tolerance MAX=([0-9\-+.eE]+)} [tolerance $e] dummy tol
    puts "Error: edge $e (lenth=$len, tolerance=$tol) is too small"
  }
}
TagsNo tags attached.
Test case numberbugs iges bug28694_1, bug28694_2, bug28694_3, bug28694_4, bug28694_5, bug28694_6, bug28694_7, bug28694_8, bug28694_9
Attached Files

- Relationships

-  Notes
(0065573)
msv (developer)
2017-04-26 18:25

The same bug is in test case "bugs iges buc60823". The command to read the model:
igesbrep [locate_data_file buc60823.igs] r *
(0065596)
msv (developer)
2017-04-27 12:37
edited on: 2017-04-28 15:04

More test cases with the same issue:
bugs iges buc60850
bugs iges bug58_1
bugs iges bug131_7
bugs iges bug133_1
bugs iges bug272_1
bugs iges bug272_2
bugs iges bug283
bugs iges bug285
bugs iges bug306
bugs iges bug365_3
bugs iges ger61337
bugs modalg_2 bug22893

(0069987)
apv (tester)
2017-08-28 16:04

Problem described in issue is reproduced on current state of OCCT.
(0078520)
git (administrator)
2018-08-09 12:27

Branch CR28694 has been created by skl.

SHA-1: fb31ed8b9b5e60994be2fc2691f129acfe670d38


Detailed log of new commits:

Author: skl
Date: Thu Aug 9 12:26:20 2018 +0300

    0028694: IGES reader produces too small edge covered by its vertices
    
    Method IsUClosed() is changed (check distance to middle point is added).
(0078540)
git (administrator)
2018-08-09 18:17

Branch CR28694_3 has been created by skl.

SHA-1: 0a711422b34e68296b484d9bd465d6ce887c3864


Detailed log of new commits:

Author: skl
Date: Thu Aug 9 18:15:41 2018 +0300

    0028694: IGES reader produces too small edge covered by its vertices
    
    Additional check.
(0078541)
git (administrator)
2018-08-09 18:21

Branch CR28694_4 has been created by skl.

SHA-1: 7865ed24092ce6e3a2d99e9fa5076fc833368a11


Detailed log of new commits:

Author: skl
Date: Thu Aug 9 18:19:40 2018 +0300

    0028694: IGES reader produces too small edge covered by its vertices
    
    Additional check.
(0078548)
git (administrator)
2018-08-10 08:15

Branch CR28694_5 has been created by skl.

SHA-1: b41288042396c1a95d9ecf9e2bc718e6808ee017


Detailed log of new commits:

Author: skl
Date: Fri Aug 10 08:14:04 2018 +0300

    0028694: IGES reader produces too small edge covered by its vertices
    
    Additional check 3.
(0078583)
git (administrator)
2018-08-13 11:03

Branch CR28694 has been updated forcibly by skl.

SHA-1: 5d0c15529aeb41cb9fd0c069979467c3fef99ef3
(0078600)
skl (developer)
2018-08-14 15:48

Branch CR28694 is ready to be reviewed.

Results of tests:
http://jenkins-test-12.nnov.opencascade.com/view/CR28694-master-skl/view/COMPARE/ [^]
(0078608)
gka (developer)
2018-08-14 18:00

Please update method ShapeAnalysis_Surface::IsUClosed and ShapeAnalysis_Surface::IsVClosed to remove redundant sqrt from values of distances
(0078612)
git (administrator)
2018-08-14 18:43

Branch CR28694 has been updated forcibly by skl.

SHA-1: cc8a2630d3578d4ab0ca4541cccd578cbddbaddd
(0078628)
git (administrator)
2018-08-15 15:35

Branch CR28694 has been updated forcibly by skl.

SHA-1: a5a9b60f1ddc86f0c97e63c1588b967a558368c6
(0078636)
skl (developer)
2018-08-15 18:28

Branch CR28694 is ready to be reviewed.

Results of tests:
http://jenkins-test-12.nnov.opencascade.com/view/CR28694-master-skl/view/COMPARE/ [^]
(0078637)
gka (developer)
2018-08-15 18:56

Please note that it is not desirable to change formatting of files during implementation of the fixes. Changing formatting should be performed as separate fix.

Please put only made modification in the class ShapeAnalysis_Surface without additional changing of formatting.

- Issue History
Date Modified Username Field Change
2017-04-26 18:10 msv New Issue
2017-04-26 18:10 msv Assigned To => gka
2017-04-26 18:25 msv Note Added: 0065573
2017-04-27 12:36 msv Relationship added related to 0023731
2017-04-27 12:37 msv Note Added: 0065596
2017-04-27 17:14 msv Note Edited: 0065596 View Revisions
2017-04-28 15:04 msv Note Edited: 0065596 View Revisions
2017-07-27 11:15 abv Target Version 7.2.0 => 7.4.0*
2017-08-28 16:04 apv Test case number => bugs iges bug28694_1, bug28694_2, bug28694_3, bug28694_4, bug28694_5, bug28694_6, bug28694_7, bug28694_8, bug28694_9
2017-08-28 16:04 apv Note Added: 0069987
2018-08-01 18:25 gka Assigned To gka => skl
2018-08-01 18:25 gka Status new => assigned
2018-08-09 12:27 git Note Added: 0078520
2018-08-09 18:17 git Note Added: 0078540
2018-08-09 18:21 git Note Added: 0078541
2018-08-10 08:15 git Note Added: 0078548
2018-08-13 11:03 git Note Added: 0078583
2018-08-14 15:48 skl Note Added: 0078600
2018-08-14 15:48 skl Assigned To skl => gka
2018-08-14 16:13 skl Status assigned => resolved
2018-08-14 18:00 gka Note Added: 0078608
2018-08-14 18:00 gka Assigned To gka => skl
2018-08-14 18:00 gka Status resolved => assigned
2018-08-14 18:43 git Note Added: 0078612
2018-08-15 15:35 git Note Added: 0078628
2018-08-15 18:28 skl Note Added: 0078636
2018-08-15 18:28 skl Assigned To skl => gka
2018-08-15 18:28 skl Status assigned => resolved
2018-08-15 18:56 gka Note Added: 0078637
2018-08-15 18:57 gka Assigned To gka => skl
2018-08-15 18:57 gka Status resolved => assigned


Copyright © 2000 - 2018 MantisBT Team
Powered by Mantis Bugtracker