View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0028780 | Community | OCCT:Modeling Algorithms | public | 2017-05-23 18:32 | 2017-09-29 16:29 |
| Reporter | guidovanhilst | Assigned To | bugmaster | ||
| Priority | normal | Severity | block | ||
| Status | closed | Resolution | fixed | ||
| Platform | Windows | OS | VC++ 2013 | ||
| Product Version | Unscheduled | ||||
| Target Version | 7.2.0 | Fixed in Version | 7.2.0 | ||
| Summary | 0028780: HLR with focus does not seem to work | ||||
| Description | If you set the focus in HLRAlgo_Projector then HLRBRep_Algo is producing very strange outlines. If focus not set (perspective=false) it seems to work good. See discussion on Forum: https://www.opencascade.com/content/perspective-projection#comment-19823 | ||||
| Steps To Reproduce | See: http://www.creativecadtechnology.com/OCC/ShowScript?userName=bugreport&groupName=demo&scriptName=HiddenLine | ||||
| Tags | No tags attached. | ||||
| Test case number | bugs modalg_7 bug28780 | ||||
|
|
Branch CR28780 has been created by abv. SHA-1: 395cb3eb910b28d4f7acf614afd615d8c804af3d Detailed log of new commits: Author: abv Date: Fri Aug 25 16:23:05 2017 +0300 0028780: HLR with focus does not seem to work Method HLRBRep_Curve::D1() is corrected to apply transformation to a point according to current projection parameters |
|
|
The fix proposed by Adriana Costas is pushed to branch CR28780, please review. Note that methods D1 and D2 still apply transformations differently; this is suspicious. |
|
|
Branch CR28780 has been updated by msv. SHA-1: caccfa49cbe8cb2bd2fe93c1ed28c476099db86e Detailed log of new commits: Author: msv Date: Fri Aug 25 20:06:47 2017 +0300 # test case for the bug |
|
|
Apparently D1 and D2 do the same thing by different ways, and both of them are correct. I have created a new test case and run Jenkins job CR28780-master-MSV. |
|
|
Reviewed. Test case has been created. Dear bugmaster, please validate the result of tests in jenkins job and integrate. |
|
|
Combination - OCCT branch : CR28780 - SHA-1: caccfa49cbe8cb2bd2fe93c1ed28c476099db86e Products branch : master was compiled on Linux, MacOS and Windows platforms and tested on optimize mode. http://jenkins-test-10.nnov.opencascade.com/view/CR28780-master-MSV/ Number of compiler warnings: OCCT : Linux: 0 (0 on master) Windows: 0 (0 on master) MacOS : 0 (0 on master) Products : Linux: 4 (4 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 |
|
|
Branch CR28780 has been deleted by kgv. SHA-1: caccfa49cbe8cb2bd2fe93c1ed28c476099db86e |
|
occt: master 2111e967 2017-08-25 13:23:05 Committer: bugmaster Details Diff |
0028780: HLR with focus does not seem to work Method HLRBRep_Curve::D1() is corrected to apply transformation to a point according to current projection parameters |
Affected Issues 0028780 |
|
| mod - src/HLRBRep/HLRBRep_Curve.cxx | Diff File | ||
| add - tests/bugs/modalg_7/bug28780 | Diff File | ||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2017-05-23 18:32 | guidovanhilst | New Issue | |
| 2017-05-23 18:32 | guidovanhilst | Assigned To | => msv |
| 2017-05-23 18:39 | guidovanhilst | Steps to Reproduce Updated | |
| 2017-05-23 18:40 | guidovanhilst | Description Updated | |
| 2017-05-23 18:40 | guidovanhilst | Steps to Reproduce Updated | |
| 2017-07-21 11:16 | msv | Target Version | 7.2.0 => 7.3.0 |
| 2017-08-25 16:23 | git | Note Added: 0069923 | |
| 2017-08-25 16:25 | abv | Note Added: 0069924 | |
| 2017-08-25 16:25 | abv | Status | new => resolved |
| 2017-08-25 16:40 | kgv | Target Version | 7.3.0 => 7.2.0 |
| 2017-08-25 20:06 | git | Note Added: 0069935 | |
| 2017-08-25 20:11 | msv | Note Added: 0069936 | |
| 2017-08-28 09:43 | msv | Note Added: 0069963 | |
| 2017-08-28 09:43 | msv | Assigned To | msv => bugmaster |
| 2017-08-28 09:43 | msv | Status | resolved => reviewed |
| 2017-08-28 09:55 | bugmaster | Note Added: 0069964 | |
| 2017-08-28 09:55 | bugmaster | Status | reviewed => tested |
| 2017-08-28 09:55 | bugmaster | Test case number | => bugs modalg_7 bug28780 |
| 2017-08-31 18:37 | bugmaster | Changeset attached | => occt master 2111e967 |
| 2017-08-31 18:37 | bugmaster | Status | tested => verified |
| 2017-08-31 18:37 | bugmaster | Resolution | open => fixed |
| 2017-09-04 18:20 | git | Note Added: 0070187 | |
| 2017-09-29 16:17 |
|
Fixed in Version | => 7.2.0 |
| 2017-09-29 16:29 |
|
Status | verified => closed |
