View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0028913 | Open CASCADE | OCCT:Modeling Algorithms | public | 2017-07-12 10:46 | 2017-09-29 16:29 |
| Reporter | imn | Assigned To | bugmaster | ||
| Priority | normal | Severity | minor | ||
| Status | closed | Resolution | fixed | ||
| Target Version | 7.2.0 | Fixed in Version | 7.2.0 | ||
| Summary | 0028913: UnifySameDomain crashes with error: "a shape is modified and removed simultaneously" | ||||
| Description | When trying to unify the attached face the algorithm crashes with error: "a shape is modified and removed simultaneously" in "BRepTools_History.cxx" | ||||
| Steps To Reproduce | restore face.brep b explode b E unifysamedom result b b_6 b_9 b_11 testgrid heal unify_same_domain | ||||
| Tags | No tags attached. | ||||
| Test case number | heal unify_same_domain A6 A7 A8 A9 B1 B2 B3 B4 B5 | ||||
|
|
face.brep (23,965 bytes) |
|
|
It is needed to decide what to do with this issue. One possible solution is to use map of back links in unify same domain algorithm. Or, may be it is needed to implement more intelligent behavior of Remove() method of History, what will require adding back map in the History. |
|
|
Up. Alexander, please take care of this. |
|
|
It is decided to overcome the problem inside UnifySameDomain algorithm. History is to be remained untouched. |
|
|
The problem is not reproduced on current version of OCCT. Please create the test case and close the issue. |
|
|
The problem is reproduced in Debug mode. |
|
|
Branch CR28913 has been created by emv. SHA-1: bcc1b7a66096505c026f88ce67bf7200b5b506ba Detailed log of new commits: Author: emv Date: Fri Aug 18 16:00:07 2017 +0300 0028913: UnifySameDomain crashes with error: "a shape is modified and removed simultaneously" Correct filling of the history of the modifications of the shapes in ShapeUpgrade_UnifySameDomain algorithm. Test cases for the issue. |
|
|
Dear Mikhail, could you please review the git branch CR28913? Test results are available here - http://jenkins-test-10.nnov.opencascade.com/view/CR28913-master-EMV/view/COMPARE/. |
|
|
Reviewed. |
|
|
Combination - OCCT branch : CR28913 : SHA-1: bcc1b7a66096505c026f88ce67bf7200b5b506ba Products branch : master was compiled on Linux, MacOS and Windows platforms and tested on optimize mode. Number of compiler warnings: OCCT : Linux: 0 (0 on master) Windows: 0 (0 on master) MacOS : 0 (0 on master) Products : Linux: 5 (5 on master) Windows: 0 (0 on master) MacOS : 0 (0 on master) Regressions/Differences/Improvements: No regressions/differences CPU differences: Windows: OCCT Total CPU difference: 17330.010689098726 / 17295.75286949859 [+0.20%] Producst Total CPU difference: 7688.758886599963 / 7720.302288799964 [-0.41%] Linux: OCCT Total CPU difference: 19451.89000000023 / 19456.560000000376 [-0.02%] Products Total CPU difference: 7703.40000000009 / 7725.120000000072 [-0.28%] Image differences : No differences that require special attention Memory differences : No differences that require special attention New test cases : OK |
|
|
Branch CR28913 has been deleted by kgv. SHA-1: bcc1b7a66096505c026f88ce67bf7200b5b506ba |
|
occt: master f523acf0 2017-08-18 13:00:07
Committer: bugmaster Details Diff |
0028913: UnifySameDomain crashes with error: "a shape is modified and removed simultaneously" Correct filling of the history of the modifications of the shapes in ShapeUpgrade_UnifySameDomain algorithm. Test cases for the issue. |
Affected Issues 0028913 |
|
| mod - src/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.cxx | Diff File | ||
| mod - src/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.hxx | Diff File | ||
| mod - tests/heal/unify_same_domain/A4 | Diff File | ||
| add - tests/heal/unify_same_domain/A6 | Diff File | ||
| add - tests/heal/unify_same_domain/A7 | Diff File | ||
| add - tests/heal/unify_same_domain/A8 | Diff File | ||
| add - tests/heal/unify_same_domain/A9 | Diff File | ||
| add - tests/heal/unify_same_domain/B1 | Diff File | ||
| add - tests/heal/unify_same_domain/B2 | Diff File | ||
| add - tests/heal/unify_same_domain/B3 | Diff File | ||
| add - tests/heal/unify_same_domain/B4 | Diff File | ||
| add - tests/heal/unify_same_domain/B5 | Diff File | ||
| add - tests/heal/unify_same_domain/end | Diff File | ||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2017-07-12 10:46 | imn | New Issue | |
| 2017-07-12 10:46 | imn | Assigned To | => msv |
| 2017-07-12 10:46 | imn | File Added: face.brep | |
| 2017-07-12 10:47 | imn | Relationship added | related to 0028245 |
| 2017-07-12 12:37 |
|
Note Added: 0068207 | |
| 2017-07-12 12:37 |
|
Assigned To | msv => abk |
| 2017-07-12 12:37 |
|
Status | new => assigned |
| 2017-07-21 10:15 |
|
Note Added: 0068526 | |
| 2017-07-21 17:56 |
|
Note Added: 0068598 | |
| 2017-07-21 17:56 |
|
Assigned To | abk => imn |
| 2017-07-21 17:56 |
|
Status | assigned => feedback |
| 2017-08-16 11:46 |
|
Assigned To | imn => bugmaster |
| 2017-08-16 11:47 |
|
Note Added: 0069439 | |
| 2017-08-16 13:28 |
|
Note Added: 0069449 | |
| 2017-08-16 13:28 |
|
Assigned To | bugmaster => imn |
| 2017-08-16 13:28 |
|
Status | feedback => assigned |
| 2017-08-16 16:05 |
|
Assigned To | imn => emv |
| 2017-08-21 16:04 |
|
Relationship added | related to 0029027 |
| 2017-08-23 10:50 | git | Note Added: 0069767 | |
| 2017-08-23 15:16 |
|
Note Added: 0069791 | |
| 2017-08-23 15:16 |
|
Assigned To | emv => msv |
| 2017-08-23 15:16 |
|
Status | assigned => resolved |
| 2017-08-23 15:16 |
|
Steps to Reproduce Updated | |
| 2017-08-23 17:13 |
|
Note Added: 0069799 | |
| 2017-08-23 17:13 |
|
Assigned To | msv => bugmaster |
| 2017-08-23 17:13 |
|
Status | resolved => reviewed |
| 2017-08-24 11:22 | bugmaster | Test case number | => heal unify_same_domain A6 A7 A8 A9 |
| 2017-08-24 11:23 | bugmaster | Test case number | heal unify_same_domain A6 A7 A8 A9 => heal unify_same_domain A6 A7 A8 A9 B1 B2 B3 B4 B5 |
| 2017-08-24 11:25 | bugmaster | Note Added: 0069818 | |
| 2017-08-24 11:25 | bugmaster | Status | reviewed => tested |
| 2017-08-25 15:02 | bugmaster | Changeset attached | => occt master f523acf0 |
| 2017-08-25 15:02 | bugmaster | Status | tested => verified |
| 2017-08-25 15:02 | bugmaster | Resolution | open => fixed |
| 2017-08-25 16:13 | git | Note Added: 0069900 | |
| 2017-09-29 16:17 |
|
Fixed in Version | => 7.2.0 |
| 2017-09-29 16:29 |
|
Status | verified => closed |
