View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0023771 | Open CASCADE | OCCT:Data Exchange | public | 2013-02-19 16:48 | 2013-12-19 13:55 |
| Reporter | Assigned To | ika | |||
| Priority | normal | Severity | minor | ||
| Status | closed | Resolution | fixed | ||
| Platform | Windows | OS | VC++ 2008 | ||
| Product Version | 6.5.4 | ||||
| Target Version | 6.7.0 | Fixed in Version | 6.7.0 | ||
| Summary | 0023771: Writing offset-based surfaces of revolution to IGES | ||||
| Description | It seems like surfaces of revolution built on offset curves are not correctly written to IGES. At least, such applications as "Rhinoceros 5" and "Creo Elements" from PTC cannot handle them: such faces are just ignored. Steps to reproduce: pload ALL test_offset brepiges f1 f1.igs brepiges f2 f2.igs Here, "test_offset" is a custom command constructing two faces of revolution: one is built on explicitly defined circular curve, another -- on Geom2d_OffsetCurve. The second face (f2) cannot be read by the mentioned external software. However, it is successfully translated back from IGES by OCCT itself. | ||||
| Tags | No tags attached. | ||||
| Test case number | bugs xde bug23771_1 bug23771_2 | ||||
|
2013-02-19 16:48 developer |
TestOffset.cpp (1,657 bytes) |
|
2013-02-19 16:48 developer |
001.png (66,267 bytes) |
|
2013-02-19 16:48 developer |
002.png (43,536 bytes) |
|
|
Dear SSV, the problem is in offset curves. OCCT writes it correctly, according IGES 5.3, but both "Rhinoceros" and "Creo Elements" can't read it. It seems like these tools haven't such ability. |
|
|
Dear IKA, Thank you for the analysis! Dear BugMaster, As there are no problems in OCCT, I propose to close this issue. |
|
|
"Rhinoceros" and "Creo Elements" can't write offset curves by themselves too. They write it like BSplines. |
|
|
I suggest to add a parameter in IGES translator to write offset curves as b-splines (yes by default), to make sure that the files written by OCCT are readable by other applications (even if they do not support full IGES spec). |
|
|
Parameter to write offset curves like b-splines was added. Branch CR23771 is ready to be reviewed. Dear GKA, Please review. |
|
|
Branch CR23771 is ready to be reviewed. |
|
|
Perhaps the default should not change. Other applications that expect and can handle native types (i.e. offsets) should not be affected. |
|
|
The fact is, quite popular Rhino and Creo do not read those offsets. We should prefer default options which allows data to be correctly translated in most real-user cases, even if some better translation can happen in specific cases. |
|
|
Dear BugMaster, Branch CR23771 (and products from GIT master) was compiled on Linux and Windows platforms and tested without rebase. SHA-1: 2388d7034f9e9a226eb5c8a5fad3f66e585256d0 Number of compiler warnings: occt component : Linux: 2 (2 on master) Windows: 11 (11 on master) products component : Linux: 0 (0 on master) Windows: 64 (64 on master) Regressions: No regressions Improvements: No improvements Testing cases: bugs xde bug23771_1 bug23771_2 - OK Testing on Linux: Total MEMORY difference: 366303552 / 366120428 Total CPU difference: 43886.880000000994 / 43868.170000001046 Testing on Windows: Total MEMORY difference: 419513360 / 421421640 Total CPU difference: 33103.125 / 32293.78125 There are not serious differences in images found by testdiff. |
|
occt: master 1939140c 2013-05-16 07:14:51 Details Diff |
0023771: Writing offset-based surfaces of revolution to IGES Parameter to write offset curves like b-splines was added Change function, which convert offset curves to b-splines Added test command "test_offset" Added test cases bugs/xde/bug23771_1 bug23771_2 |
Affected Issues 0023771 |
|
| mod - src/GeomToIGES/GeomToIGES_GeomCurve.cxx | Diff File | ||
| mod - src/IGESData/IGESData.cxx | Diff File | ||
| mod - src/QABugs/QABugs_19.cxx | Diff File | ||
| add - tests/bugs/xde/bug23771_1 | Diff File | ||
| add - tests/bugs/xde/bug23771_2 | Diff File | ||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2013-02-19 16:48 |
|
New Issue | |
| 2013-02-19 16:48 |
|
Assigned To | => gka |
| 2013-02-19 16:48 |
|
File Added: TestOffset.cpp | |
| 2013-02-19 16:48 |
|
File Added: 001.png | |
| 2013-02-19 16:48 |
|
File Added: 002.png | |
| 2013-04-08 10:21 |
|
Assigned To | gka => ika |
| 2013-04-08 10:21 |
|
Status | new => assigned |
| 2013-04-12 16:02 | ika | Note Added: 0024120 | |
| 2013-04-12 16:02 | ika | Assigned To | ika => ssv |
| 2013-04-12 16:02 | ika | Status | assigned => feedback |
| 2013-04-12 16:34 |
|
Assigned To | ssv => |
| 2013-04-12 16:34 |
|
Assigned To | => bugmaster |
| 2013-04-12 16:34 |
|
Note Added: 0024122 | |
| 2013-04-12 17:18 | ika | Note Added: 0024124 | |
| 2013-04-12 17:19 |
|
Note Added: 0024125 | |
| 2013-04-12 17:19 |
|
Assigned To | bugmaster => ika |
| 2013-04-12 17:19 |
|
Status | feedback => assigned |
| 2013-04-16 14:30 | ika | Note Added: 0024155 | |
| 2013-04-16 14:30 | ika | Assigned To | ika => gka |
| 2013-04-16 14:30 | ika | Status | assigned => resolved |
| 2013-04-16 16:05 |
|
Note Added: 0024163 | |
| 2013-04-16 16:05 |
|
Status | resolved => reviewed |
| 2013-04-17 18:41 | apn | Assigned To | gka => apn |
| 2013-05-08 15:32 | apn | Test case number | => bugs xde bug23771_1 bug23771_2 |
| 2013-05-10 15:41 | Roman Lygin | Note Added: 0024357 | |
| 2013-05-11 10:44 |
|
Note Added: 0024360 | |
| 2013-05-13 15:14 | apn | Note Added: 0024369 | |
| 2013-05-13 15:14 | apn | Assigned To | apn => bugmaster |
| 2013-05-13 15:14 | apn | Status | reviewed => tested |
| 2013-05-20 10:56 | ika | Changeset attached | => occt master 1939140c |
| 2013-05-20 10:56 | ika | Assigned To | bugmaster => ika |
| 2013-05-20 10:56 | ika | Status | tested => verified |
| 2013-05-20 10:56 | ika | Resolution | open => fixed |
| 2013-05-20 16:53 | bugmaster | Target Version | => 6.7.0 |
| 2013-12-19 13:53 | bugmaster | Status | verified => closed |
| 2013-12-19 13:55 | bugmaster | Fixed in Version | => 6.7.0 |
