View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0023974 | Open CASCADE | OCCT:Data Exchange | public | 2013-05-20 17:01 | 2018-09-09 13:06 |
| Reporter | Assigned To | bugmaster | |||
| Priority | normal | Severity | minor | ||
| Status | closed | Resolution | fixed | ||
| Target Version | 7.4.0 | Fixed in Version | 7.4.0 | ||
| Summary | 0023974: Optimization of convertation to STEP. | ||||
| Description | Optimization of convertation to STEP. In order to improve performance it is necessary to reduce number of call function Transfer_Binder::AddResult | ||||
| Tags | No tags attached. | ||||
| Test case number | Not needed | ||||
|
|
Attached archive contains 2 scripts: t1 and t3. Each script opens BREP file and convert it to STEP format. First script convert 1000 prisms, t3 -- 3000. Initially t1 was executed during 45 seconds, t3 -- 3m 45s. After modification: t1 -- 25s, t3 -- 50s. |
|
2013-05-21 14:27 developer |
Scripts.7z (318,791 bytes) |
|
|
I have tested the fix on current maater; my results on t1 is: 7.5 sec on master, 2.7 sec with the fix. There are two problems however (both on master and with the fix): - if I add 'newmodel' before 'stepwrite', test becomes much longer (35 and 26 sec, respectively) - calling 'newmodel' after the test (or repeating it twice) kills DRAW |
|
|
See message of user Timo on OCCT forum, which can be related to similar problem: http://www.opencascade.org/org/forum/thread_27183/?forum=3 When STEPControl_Writer::Transfer () is called multiple times in order to export a list of shapes to a common STEP file, the performance might be bad because the third parameter "compgraph" of the Transfer function is true by default. It means after every translation of a shape the graph is rebuilt. Instead this parameter can be set to false and only after all shapes are translated the graph can be built once only by: writer.WS()->ComputeGraph(Standard_True ); |
|
|
The patch for 0029830 fixes this issue. The following timings for scripts: t1 - 2.7 s t3 - 8 s There is no need to create such test case, as the patch 29830 contains the similar test cases where a dozens of shapes are put in the same compound and written to STEP model. |
|
|
The patch for 29830 also contains protection agains stack overflow during destruction of such model. |
|
|
Please close this issue along with 0029830. |
|
|
Branch CR23974 has been deleted by inv. SHA-1: 52f17edcc6bb128c061587f52b3cfb9efdd2a730 |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2013-05-20 17:01 |
|
New Issue | |
| 2013-05-20 17:01 |
|
Assigned To | => gka |
| 2013-05-20 17:02 |
|
Assigned To | gka => sdv |
| 2013-05-21 14:13 |
|
Note Added: 0024450 | |
| 2013-05-21 14:27 |
|
File Added: Scripts.7z | |
| 2013-05-21 17:53 |
|
Relationship added | related to 0023979 |
| 2013-05-21 18:29 |
|
Assigned To | sdv => gka |
| 2013-05-21 18:29 |
|
Status | new => resolved |
| 2014-04-04 17:11 |
|
Target Version | => 6.8.0 |
| 2014-09-26 18:27 |
|
Note Added: 0032266 | |
| 2014-09-26 18:28 |
|
Note Edited: 0032266 | |
| 2014-10-17 02:36 |
|
Target Version | 6.8.0 => 7.1.0 |
| 2015-06-09 17:46 |
|
Note Added: 0042065 | |
| 2016-10-26 12:06 |
|
Target Version | 7.1.0 => 7.2.0 |
| 2017-08-15 12:42 |
|
Target Version | 7.2.0 => 7.4.0 |
| 2018-06-09 12:48 |
|
Assigned To | gka => msv |
| 2018-06-09 15:01 |
|
Relationship added | related to 0029830 |
| 2018-06-09 15:10 |
|
Note Added: 0076696 | |
| 2018-06-09 15:11 |
|
Note Added: 0076697 | |
| 2018-06-09 18:21 |
|
Note Added: 0076712 | |
| 2018-06-09 18:21 |
|
Assigned To | msv => bugmaster |
| 2018-06-09 18:21 |
|
Status | resolved => feedback |
| 2018-08-28 13:00 | bugmaster | Status | feedback => verified |
| 2018-08-28 13:00 | bugmaster | Resolution | open => fixed |
| 2018-09-03 16:55 | bugmaster | Test case number | => Not needed |
| 2018-09-09 13:06 | git | Note Added: 0079058 |
