MantisBT - Community
View Issue Details
0024595Community[OCCT] OCCT:Data Exchangepublic2014-02-04 04:592016-04-20 15:50
thomhong 
bugmaster 
normalminor 
closedfixed 
[OCCT] 6.7.0 
[OCCT] 7.0.0[OCCT] 7.0.0 
bugs step bug24595
0024595: STEP import missing surfaces
The imported model is missing surfaces at the bottom (lowest Z) of the model. The bottom end of the model should be spherical and closed, not open.
1. Open the attached STEP file (using ImportExport sample project)

//==================================
test script bug24595 was added.
No tags attached.
related to 0023202closed abv Community BRepMesh of face fails 
? 281390.stp (431,098) 2014-02-04 04:59
https://tracker.dev.opencascade.org/
? 280696.stp (182,116) 2014-08-21 02:09
https://tracker.dev.opencascade.org/
? 280612.stp (909,537) 2014-08-21 20:26
https://tracker.dev.opencascade.org/
? bug24595 (821) 2015-02-12 15:29
https://tracker.dev.opencascade.org/
Issue History
2014-02-04 04:59thomhongNew Issue
2014-02-04 04:59thomhongAssigned To => gka
2014-02-04 04:59thomhongFile Added: 281390.stp
2014-02-04 06:57abvNote Added: 0027743
2014-02-04 06:57abvNote Edited: 0027743bug_revision_view_page.php?bugnote_id=27743#r6705
2014-02-04 20:52thomhongNote Added: 0027754
2014-08-21 02:09thomhongFile Added: 280696.stp
2014-08-21 02:14thomhongNote Added: 0030964
2014-08-21 20:26thomhongFile Added: 280612.stp
2014-09-11 18:32abvTarget Version => 7.1.0
2015-02-06 17:08gkaAssigned Togka => akz
2015-02-06 17:08gkaStatusnew => assigned
2015-02-11 20:44gitNote Added: 0037393
2015-02-12 15:29akzFile Added: bug24595
2015-02-12 15:30akzNote Added: 0037443
2015-02-12 15:30akzAssigned Toakz => gka
2015-02-12 15:30akzStatusassigned => resolved
2015-02-12 15:30akzSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=9360#r9360
2015-02-13 17:55gkaNote Added: 0037527
2015-02-13 17:55gkaAssigned Togka => bugmaster
2015-02-13 17:55gkaStatusresolved => reviewed
2015-02-13 18:01apvAssigned Tobugmaster => apv
2015-02-13 18:46gitNote Added: 0037537
2015-02-13 18:47apvNote Added: 0037538
2015-02-17 18:20apvNote Added: 0037638
2015-02-17 18:20apvAssigned Toapv => akz
2015-02-17 18:20apvStatusreviewed => assigned
2015-12-11 11:02gitNote Added: 0049009
2015-12-11 11:03gkaNote Added: 0049010
2015-12-11 11:03gkaAssigned Toakz => abv
2015-12-11 11:03gkaStatusassigned => resolved
2015-12-11 12:59abvNote Added: 0049020
2015-12-11 12:59abvAssigned Toabv => bugmaster
2015-12-11 12:59abvStatusresolved => reviewed
2015-12-11 14:01apvAssigned Tobugmaster => apv
2015-12-11 15:27gitNote Added: 0049047
2015-12-11 15:28apvNote Added: 0049048
2015-12-15 12:34apvNote Added: 0049137
2015-12-15 12:34apvAssigned Toapv => gka
2015-12-15 12:34apvStatusreviewed => assigned
2015-12-15 12:35apvNote Added: 0049138
2015-12-15 17:11gkaAssigned Togka => apv
2015-12-15 17:12gkaNote Added: 0049170
2015-12-15 17:52gkaStatusassigned => feedback
2015-12-16 16:21bugmasterAssigned Toapv => bugmaster
2015-12-16 16:21bugmasterStatusfeedback => tested
2015-12-17 11:42gitNote Added: 0049224
2015-12-17 12:09gitNote Added: 0049226
2015-12-17 12:10apvTest case number => bugs step bug24595
2015-12-17 12:11apvNote Added: 0049227
2015-12-17 12:12apvNote Edited: 0049227bug_revision_view_page.php?bugnote_id=49227#r12577
2015-12-18 15:25bugmasterChangeset attached => occt master 4f99f761
2015-12-18 15:25bugmasterStatustested => verified
2015-12-18 15:25bugmasterResolutionopen => fixed
2015-12-21 12:53bugmasterTarget Version7.1.0 => 7.0.0
2016-01-29 16:14oanRelationship addedrelated to 0027122
2016-02-15 15:17abvRelationship addedrelated to 0023202
2016-04-17 13:54gitNote Added: 0052975
2016-04-17 13:54gitNote Added: 0052976
2016-04-20 15:43aivFixed in Version => 7.0.0
2016-04-20 15:50aivStatusverified => closed

Notes
(0027743)
abv   
2014-02-04 06:57   
The face (# 74) is present but incorrectly oriented (it is based on spherical surface and wrong part of the surface is taken). OCCT translator does not report any warnings on this face, that suggests that the face might be incorrectly encoded in the file. Do other translators (CAD systems etc.) open this file correctly?

(0027754)
thomhong   
2014-02-04 20:52   
Thank you for investigating this issue for us. I believe that the file opens correctly in NX, as well as IDA-STEP (http://www.ida-step.net/components/viewers/basic [^]).

I will pass on your comment to the author of the STEP file.
(0030964)
thomhong   
2014-08-21 02:14   
Uploaded another example today with the same or similar issue. Also opens fine in other CAD systems (IDA-STEP, DelCam Exchange 2013, CT Core Tech's 3D_Evolution 2014, DP Esprit 2014), but bottom surface import has issue in OpenCASCADE's ImportExport sample.
(0037393)
git   
2015-02-11 20:44   
Branch CR24595 has been created by akz.

SHA-1: 52869cd0cc257f90f85bd98a2b9996e7b6c6f756


Detailed log of new commits:

Author: akz
Date: Wed Feb 11 20:40:25 2015 +0300

    0024595: STEP import missing surfaces
    
    According to the specification of ISO-10303 part 42:
    "If the face has only one bound and this is of type vertex_loop, then the interior of the face is the domain of the face_surface.face_geometry. In such a case the underlying surface shall be closed (e.g. a spherical_surface.)"
    - natural bounds are applied only in case if VertexLoop is only the one
      defined face bound.
(0037443)
akz   
2015-02-12 15:30   
Branch CR24595 is ready to review
(0037527)
gka   
2015-02-13 17:55   
Branch CR24595 is ready to be tested
(0037537)
git   
2015-02-13 18:46   
Branch CR24595 has been updated forcibly by apv.

SHA-1: ea2cf441b741a94cf7ac97a5894f959e68128752
(0037538)
apv   
2015-02-13 18:47   
Branch CR24595 has been rebased on the current master
(0037638)
apv   
2015-02-17 18:20   
Dear BugMaster,

Branch CR24595 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
SHA-1: ea2cf441b741a94cf7ac97a5894f959e68128752

Number of compiler warnings:
occt component:
   Linux: 18 (18 on master)
   Windows: 0 (0 on master)
products component:
   Linux: 11 (11 on master)
   Windows: 1 (1 on master)

Regressions/Differences:
http://occt-tests/CR24595-master-occt/Debian60-64/summary.html [^]
http://occt-tests/CR24595-master-occt/Windows-32-VC10/summary.html [^]
de step_1(004) D1, X2
de step_2(005) S5

Testing on Linux:
Total MEMORY difference: 86046407 / 86143565
Total CPU difference: 48119.38000000016 / 48438.35999999994

Testing on Windows:
Total MEMORY difference: 40684398 / 40695157
Total CPU difference: 36003.75 / 34792.25
(0049009)
git   
2015-12-11 11:02   
Branch CR24595_1 has been created by gka.

SHA-1: 84e82ff782ccd0186281b0eefad7cf1c4baad5fc


Detailed log of new commits:

Author: gka
Date: Fri Dec 11 11:01:27 2015 +0300

    0024595: STEP import missing surfaces
    
    Modification to create natural bounds for face based on the spherical and Bspline surface and having only one bound represented by Vertex loop was made.
     According to the specification of ISO-10303 part 42:
        "If the face has only one bound and this is of type vertex_loop, then the interior of the face is the domain of the face_surface.face_geometry. In such a case the underlying surface shall be closed (e.g. a spherical_surface.)"
        - natural bounds are applied only in case if VertexLoop is only the one
          defined face bound.
(0049010)
gka   
2015-12-11 11:03   
Branch CR24595_1 is ready to be reviewed
(0049020)
abv   
2015-12-11 12:59   
No remarks, please test
(0049047)
git   
2015-12-11 15:27   
Branch CR24595_1 has been updated forcibly by apv.

SHA-1: 9ef79feaed83e7a92d4e0a0cdd8331348f5881bf
(0049048)
apv   
2015-12-11 15:28   
Branch CR24595_1 has been rebased on the current master
(0049137)
apv   
2015-12-15 12:34   
Dear BugMaster,

Branch CR24595_1 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
SHA-1: 9ef79feaed83e7a92d4e0a0cdd8331348f5881bf

Number of compiler warnings:
occt component:
   Linux: 0 (0 on master)
   Windows: 0 (0 on master)
   MacOS: 134 (134 on master)
products component:
   Linux: 37 (37 on master)
   Windows: 0 (0 on master)

Regressions/Differences:
http://occt-tests/CR24595-1-master-occt-64/Windows-64-VC10/summary.html [^]
de step_3 D8

Testing on Linux:
Total MEMORY difference: 93338926 / 93828350 [-0.52%]
Total CPU difference: 18935.819999999523 / 19236.40999999988 [-1.56%]

Testing on Windows:
Total MEMORY difference: 56185248 / 56187082 [-0.00%]
Total CPU difference: 17551.532109098946 / 17780.338775798962 [-1.29%]
(0049138)
apv   
2015-12-15 12:35   
Dear gka,

Branch CR24595_1 has been rejected due to:
- regressions/differences/improvements
(0049170)
gka   
2015-12-15 17:12   
Dear apv,

This case is not regression
(0049224)
git   
2015-12-17 11:42   
Branch CR24595_1 has been updated by apv.

SHA-1: f7a5b5c7a225920dc5fb85a0181378ff6ee019a0


Detailed log of new commits:

Author: apv
Date: Thu Dec 17 11:42:07 2015 +0300

    Update of test-cases according to the new behavior

(0049226)
git   
2015-12-17 12:09   
Branch CR24595_1 has been updated by apv.

SHA-1: a1bd9892fefcd8545472da8e00aeec44214db034


Detailed log of new commits:

Author: apv
Date: Thu Dec 17 12:08:49 2015 +0300

    Test-case for issue 0024595

(0049227)
apv   
2015-12-17 12:11   
(edited on: 2015-12-17 12:12)
Testing case result:
bugs step bug24595 - OK
http://occt-tests/CR24595-1-master-occt-64/Debian70-64/bugs/step/bug24595.html [^]
http://occt-tests/CR24595-1-master-occt-64/Windows-64-VC10/bugs/step/bug24595.html [^]

(0052975)
git   
2016-04-17 13:54   
Branch CR24595 has been deleted by kgv.

SHA-1: ea2cf441b741a94cf7ac97a5894f959e68128752
(0052976)
git   
2016-04-17 13:54   
Branch CR24595_1 has been deleted by kgv.

SHA-1: a1bd9892fefcd8545472da8e00aeec44214db034