MantisBT - Open CASCADE
View Issue Details
0028144Open CASCADE[OCCT] OCCT:Modeling Datapublic2016-11-22 16:302019-09-16 09:51
epa 
bugmaster 
normalmajor 
reviewedopen 
[OCCT] 6.6.0 
[OCCT] 7.4.0 
bugs vis bug28144
0028144: Modeling Data, GeomAdaptor_Curve - regression in comparison of version OCCT654
The problem is in translation of face №128(entity 507:0000516). It looks the rage of edge is computed incorrectly (it was OK in OCCT654).

restore [locate_data_file bug28144.brep] a
axo; fit
vdisplay a; vfit
No tags attached.
child of 0028118closed abv Community Mesh generation hangs then crashes 
child of 0023062closed ika Open CASCADE Incorrect display of edge in draw axo viewer 
? 18547.stp (491,547) 2016-11-22 16:30
https://tracker.dev.opencascade.org/
png face_128.png (21,310) 2016-11-22 16:32
https://tracker.dev.opencascade.org/
png bug28144_occt730_740.png (86,096) 2019-09-04 22:38
https://tracker.dev.opencascade.org/
? bug28144.brep (23,514) 2019-09-04 22:58
https://tracker.dev.opencascade.org/
Issue History
2016-11-22 16:30epaNew Issue
2016-11-22 16:30epaAssigned To => gka
2016-11-22 16:30epaFile Added: 18547.stp
2016-11-22 16:30epaFile Added: face_128.png
2016-11-22 16:31epaFile Deleted: face_128.png
2016-11-22 16:32epaFile Added: face_128.png
2016-11-22 23:25abvRelationship addedchild of 0028118
2016-11-22 23:26abvTarget Version7.1.0 => 7.2.0
2016-11-23 11:38gkaNote Added: 0060715
2017-08-15 14:52abvTarget Version7.2.0 => 7.4.0
2017-08-23 14:24mkvTest case number => bugs step bug28144
2017-08-23 14:24mkvNote Added: 0069785
2019-09-04 22:38abvFile Added: bug28144_occt730_740.png
2019-09-04 22:57abvTest case numberbugs step bug28144 => bugs vis bug28144
2019-09-04 22:57abvNote Added: 0086757
2019-09-04 22:57abvAssigned Togka => bugmaster
2019-09-04 22:57abvStatusnew => resolved
2019-09-04 22:57abvCategoryOCCT:Data Exchange => OCCT:Visualization
2019-09-04 22:57abvSummaryStep reader - regression in comparison of version OCCT654 => Visualization - regression in comparison of version OCCT654
2019-09-04 22:57abvSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=21737#r21737
2019-09-04 22:58abvFile Added: bug28144.brep
2019-09-04 23:00gitNote Added: 0086758
2019-09-04 23:42kgvRelationship addedduplicate of 0030273
2019-09-04 23:43kgvRelationship replacedrelated to 0030273
2019-09-04 23:44kgvNote Added: 0086760
2019-09-04 23:44kgvCategoryOCCT:Visualization => OCCT:Modeling Algorithms
2019-09-04 23:44kgvSummaryVisualization - regression in comparison of version OCCT654 => Modeling Algorithms - regression in comparison of version OCCT654
2019-09-04 23:45kgvNote Edited: 0086760bug_revision_view_page.php?bugnote_id=86760#r21739
2019-09-04 23:47kgvProduct Version => 6.6.0
2019-09-04 23:49kgvRelationship addedchild of 0023062
2019-09-04 23:50kgvNote Edited: 0086760bug_revision_view_page.php?bugnote_id=86760#r21740
2019-09-04 23:51kgvNote Edited: 0086760bug_revision_view_page.php?bugnote_id=86760#r21741
2019-09-05 10:27kgvNote Added: 0086779
2019-09-05 10:28kgvCategoryOCCT:Modeling Algorithms => OCCT:Modeling Data
2019-09-05 10:28kgvSummaryModeling Algorithms - regression in comparison of version OCCT654 => Modeling Data, GeomAdaptor_Curve - regression in comparison of version OCCT654
2019-09-05 21:22abvNote Added: 0086821
2019-09-13 19:58bugmasterStatusresolved => reviewed
2019-09-14 10:57bugmasterAssigned Tobugmaster => epa
2019-09-14 10:57bugmasterStatusreviewed => assigned
2019-09-14 11:58gitNote Added: 0087083
2019-09-14 12:02abvNote Added: 0087084
2019-09-14 12:02abvAssigned Toepa => kgv
2019-09-14 12:02abvStatusassigned => resolved
2019-09-16 09:50gitNote Added: 0087110
2019-09-16 09:51kgvAssigned Tokgv => bugmaster
2019-09-16 09:51kgvStatusresolved => reviewed

Notes
(0060715)
gka   
2016-11-23 11:38   
Face 128 read from entity 507 is valid. But AXON viewer shows this face as invalid. Please see attached image.
This due to that first edge in the face based on the periodic curve has range -1.31441524331422e-011 0.214529066155427.
(0069785)
mkv   
2017-08-23 14:24   
Problem described in issue is reproduced on current state of OCCT.
(0086757)
abv   
2019-09-04 22:57   
The problem was actually not in STEP translation but in visualization of the edge with specific range. The same face (translated with current master) is shown without edge in OCCT 7.3.0 (both AXO and V3d viewers) but is OK with current master. The issue thus should be considered as fixed.

I pushed branch CR28144 containing corrected test case for this issue: it reads BREP file instead of STEP (this should be both faster and not depending on translation), and uses wireframe in 3D viewer (the problem is not visible in shading). Please add the attached BREP file to the test databsse and consider the branch for integration.
(0086758)
git   
2019-09-04 23:00   
Branch CR28144 has been created by abv.

SHA-1: 0166ba6fb7ba314f34e7fbc81d7ea70a26b91fc9


Detailed log of new commits:

Author: abv
Date: Wed Sep 4 22:57:24 2019 +0300

    0028144: Visualization - regression in comparison of version OCCT654
    
    Test case moved to grid vis and corrected to use BREP instead of STEP, and to use wireframe in 3D viewer
(0086760)
kgv   
2019-09-04 23:44   
(edited on: 2019-09-04 23:51)
> The problem was actually not in STEP translation
> but in visualization of the edge with specific range.
Sorry, I'm unable following this logic.
The fixed bug #0030273 was within GeomAdaptor_Curve::Intervals() algorithm, introduced by 0023062.

(0086779)
kgv   
2019-09-05 10:27   
> tests/bugs/vis/bug28144
Test should be moved to a proper group (moddata_X).
(0086821)
abv   
2019-09-05 21:22   
>> Sorry, I'm unable following this logic.

The logic is straightforward: I did not delve into research of the rootcause of the problem, just noticed that it is not due to STEP and problem manifests in visualization.
(0087083)
git   
2019-09-14 11:58   
Branch CR28144 has been updated forcibly by abv.

SHA-1: fcc1f590c6908b382f70cb25e9062acf842a37b1
(0087084)
abv   
2019-09-14 12:02   
Test renamed to be in the same grid as test for #30273; branch CR28144 rebased on current IR (OCCT-740beta2). Please review again
(0087110)
git   
2019-09-16 09:50   
Branch CR28144 has been updated forcibly by kgv.

SHA-1: cd58de63de485fca0c7da52c4541853af9c990d3