Anonymous | Login | 2021-01-17 19:09 MSK | ![]() |
My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||
0025654 | Open CASCADE | [OCCT] OCCT:Data Exchange | public | 2014-12-24 12:46 | 2020-09-25 20:35 | ||||||||
Reporter | abv | ||||||||||||
Assigned To | abv | ||||||||||||
Priority | normal | Severity | integration request | ||||||||||
Status | feedback | Resolution | open | ||||||||||
Platform | OS | OS Version | |||||||||||
Product Version | |||||||||||||
Target Version | [OCCT] 7.6.0* | Fixed in Version | |||||||||||
Summary | 0025654: Data Exchange - Disable writing pcurves to STEP and IGES by default | ||||||||||||
Description | Currently global DE parameter write.surfacecurve.mode is set to 1 by default, i.e. pcurves are stored in the file when it is written to STEP and IGES. Theoretically this should improve quality of translation and make import faster. However in practice it seems to be not the case. For instance, problem reported in #25523 disappears if pcurves are not written. On that model, writing without pcurves reduces size of the file 3 times, eliminates problems of loading that file to Hoops Exchange Demo, and improves loading to Pro Engineer. On some other models from customers we also observe considerable difference in size of STEP file written with pcurves vs. one without it (3.6 Mb vs. 17 Mb), with no visible effect on performance of import. Hence it will be logical to change default value of write.surfacecurve.mode to 0. | ||||||||||||
Steps To Reproduce | This bug is reproduced only when you open the step file in external CAD systems(such as ProEngineer, SolidWorks, Rhinoceros). http://tracker.dev.opencascade.org/view.php?id=25523 [^] | ||||||||||||
Tags | No tags attached. | ||||||||||||
Test case number | |||||||||||||
Attached Files | |||||||||||||
![]() |
|||||||
|
![]() |
|
(0035869) git (administrator) 2014-12-30 10:51 |
Branch CR25654 has been created by ink. SHA-1: 9b633447fb4567efafa4756818b301db9c6e0944 Detailed log of new commits: Author: ink Date: Tue Dec 30 10:50:21 2014 +0300 0025654: Disable writing pcurves to STEP and IGES by default Changed default value of write.surfacecurve.mode to 0. Added SplitClosedFaces to XSTEPResource/STEP. This is necessary for correct display of closed surfaces (such as a torus) in applications such as ProEngineer, SolidWorks, Rhinoceros. |
(0035877) abv (manager) 2014-12-30 12:46 |
Reviewed and tested in external CAD systems, please run OCCT non-regression tests and integrate. Model from #25523 is now opened fine in ProEngineer, Rhinoceros 4.0, Solidworks (there are minor artifacts however, looking like too rough mesh at the end of a pipe), and Hoops Exchange demo. Also tested on complete Stator model in CAD Assistant on Android (ASUS Nexus 7): - with pcurves: file size is 64 Mb, opened in ~ 2200 sec, 77 K faces, mesh has 1.3 M triangles - without pcurves: file size is 29 Mb, opened in ~ 1100 sec, 76 K faces, mesh has 1.5 M triangles Note that variant without pcurves is not only twice less and is loaded twice faster, but also does not fail on reading of sub-shape subject of #25523, this is a reason of having more faces and triangles. Memory usage is of the same scale (VM ~ 1400 Mb), the only different parameter is Working Set (611 Mb vs 643 Mb). |
(0036146) git (administrator) 2015-01-16 15:31 |
Branch CR25654 has been updated forcibly by mkv. SHA-1: 0d46b096c8b0e5bd923fb96651ad4ea1636bf8e4 |
(0036203) mkv (tester) 2015-01-19 14:08 |
Dear BugMaster, Branch CR25654 was rebased on current master of occt git-repository. |
(0036204) mkv (tester) 2015-01-19 14:09 |
Dear BugMaster, Branch CR25654 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: 0d46b096c8b0e5bd923fb96651ad4ea1636bf8e4 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/CR25654-master-occt/Debian60-64/summary.html [^] http://occt-tests/CR25654-master-products/Debian60-64/summary.html [^] Testing on Linux: occt component : Total MEMORY difference: 315865072 / 319325816 Total CPU difference: 35996.94000000016 / 33649.83000000009 products component : Total MEMORY difference: 108405040 / 110985888 Total CPU difference: 16655.759999999966 / 15822.40999999997 |
(0037348) git (administrator) 2015-02-11 13:16 |
Branch CR25654 has been updated by ink. SHA-1: dd4afe2fb68fefe496596aada5595841af9547b6 Detailed log of new commits: Author: ink Date: Wed Feb 11 12:53:18 2015 +0300 Tests changed in connection with changing tolerance |
(0037349) ink (developer) 2015-02-11 13:17 |
Tests changed in connection with changing tolerance(tolerance decreased). |
(0037421) git (administrator) 2015-02-12 11:56 |
Branch CR25654 has been updated forcibly by ink. SHA-1: 7f6574299e86e91aab73097fa0a393172070b54c |
(0037422) git (administrator) 2015-02-12 11:56 |
Branch CR25654 has been updated forcibly by ink. SHA-1: 499045ff88d9b046b59d94de40b19dc39b34bc3f |
(0037563) gka (developer) 2015-02-16 11:52 |
Branch CR25654 is ready to be tested |
(0037566) git (administrator) 2015-02-16 13:33 |
Branch CR25654 has been updated forcibly by mkv. SHA-1: 9729c0dd9065da7cd78698dba87abc97fb7bcc6a |
(0037601) mkv (tester) 2015-02-16 20:11 |
Dear BugMaster, Branch CR25654 was rebased on current master of occt git-repository. |
(0037602) mkv (tester) 2015-02-16 20:12 |
Dear BugMaster, Branch CR25654 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: 9729c0dd9065da7cd78698dba87abc97fb7bcc6a 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: No regressions/differences Testing cases: http://occt-tests/CR25654-master-occt/Debian60-64/summary.html [^] http://occt-tests/CR25654-master-occt/Windows-32-VC10/summary.html [^] http://occt-tests/CR25654-master-products/Debian60-64/summary.html [^] http://occt-tests/CR25654-master-products/Windows-32-VC10/summary.html [^] |
(0038875) git (administrator) 2015-03-25 12:08 |
Branch CR25654 has been updated forcibly by ink. SHA-1: 0af52d84ea78b77a44c3ad73749a61fb313db5d7 |
(0076800) kgv (developer) 2018-06-15 15:43 |
> 2015-02-16 20:12 mkv Status reviewed => assigned What is the state of this issue? I see the status change, but no comment WHY it was done (test report does not refer issues). |
![]() |
|||
Date Modified | Username | Field | Change |
2014-12-24 12:46 | abv | New Issue | |
2014-12-24 12:46 | abv | Assigned To | => gka |
2014-12-24 12:49 | abv | Relationship added | related to 0025523 |
2014-12-30 10:09 | abv | Assigned To | gka => ink |
2014-12-30 10:09 | abv | Status | new => assigned |
2014-12-30 10:51 | git | Note Added: 0035869 | |
2014-12-30 11:49 | ink | Assigned To | ink => gka |
2014-12-30 11:49 | ink | Status | assigned => resolved |
2014-12-30 11:49 | ink | Steps to Reproduce Updated | View Revisions |
2014-12-30 12:46 | abv | Note Added: 0035877 | |
2014-12-30 12:46 | abv | Assigned To | gka => bugmaster |
2014-12-30 12:46 | abv | Status | resolved => reviewed |
2015-01-16 15:31 | git | Note Added: 0036146 | |
2015-01-16 15:48 | mkv | Assigned To | bugmaster => mkv |
2015-01-19 14:08 | mkv | Note Added: 0036203 | |
2015-01-19 14:09 | mkv | Note Added: 0036204 | |
2015-01-19 14:09 | mkv | Assigned To | mkv => abv |
2015-01-19 14:09 | mkv | Status | reviewed => assigned |
2015-01-19 14:18 | abv | Assigned To | abv => ink |
2015-02-11 13:16 | git | Note Added: 0037348 | |
2015-02-11 13:17 | ink | Note Added: 0037349 | |
2015-02-12 10:13 | ink | Assigned To | ink => gka |
2015-02-12 10:13 | ink | Status | assigned => resolved |
2015-02-12 11:56 | git | Note Added: 0037421 | |
2015-02-12 11:56 | git | Note Added: 0037422 | |
2015-02-16 11:52 | gka | Note Added: 0037563 | |
2015-02-16 11:52 | gka | Assigned To | gka => bugmaster |
2015-02-16 11:52 | gka | Status | resolved => reviewed |
2015-02-16 13:07 | mkv | Assigned To | bugmaster => mkv |
2015-02-16 13:33 | git | Note Added: 0037566 | |
2015-02-16 20:11 | mkv | Note Added: 0037601 | |
2015-02-16 20:12 | mkv | Note Added: 0037602 | |
2015-02-16 20:12 | mkv | Assigned To | mkv => ink |
2015-02-16 20:12 | mkv | Status | reviewed => assigned |
2015-03-25 12:08 | git | Note Added: 0038875 | |
2015-04-14 18:15 | gka | Target Version | 6.9.0 => 7.0.0 |
2016-02-08 10:12 | abv | Target Version | 7.0.0 => 7.1.0 |
2016-10-26 20:03 | gka | Assigned To | ink => |
2016-10-26 20:03 | gka | Target Version | 7.1.0 => 7.2.0 |
2016-10-28 15:10 | Timo | Relationship added | related to 0028025 |
2017-07-27 09:33 | abv | Target Version | 7.2.0 => 7.4.0 |
2018-06-15 15:43 | kgv | Note Added: 0076800 | |
2018-06-15 15:43 | kgv | Assigned To | => abv |
2018-06-15 15:43 | kgv | Status | assigned => feedback |
2018-06-15 15:43 | kgv | Severity | minor => integration request |
2019-07-10 21:38 | abv | Target Version | 7.4.0 => 7.5.0 |
2020-03-04 11:50 | kgv | Summary | Disable writing pcurves to STEP and IGES by default => Data Exchange - Disable writing pcurves to STEP and IGES by default |
2020-09-25 20:35 | abv | Target Version | 7.5.0 => 7.6.0* |
Copyright © 2000 - 2021 MantisBT Team |