DiscussionsIssue archiveOCCT:Visualization

Archived issue #0032334

Visualization - SelectMgr_RectangularFrustum::Overlaps() does not set triangle normal in some cases

Open CASCADEOCCT:Visualizationclosed5 public notes

Search issues

Description

SelectMgr_RectangularFrustum::Overlaps() sets triangle normal to SelectBasics_PickResult only in case of internal intersection.

However, there is no reason for such selective output - computed triangle normal should be set to picking result in other cases as well.

Steps to reproduce

N/A

Public activity

5 archived notes

Participants are labeled by their role within this record.

01Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Apr 27 12:31:23 2021 +0300

    0032334: Visualization - SelectMgr_RectangularFrustum::Overlaps() does not set triangle normal in some cases
    
    Added missing SelectBasics_PickResult::SetSurfaceNormal().
02Author
Patch is ready for review
- OCCT branch: [archived branch]

http://jenkins-test-12.nnov.opencascade.com:8080/view/CR32334-master-KGV/
03Commenter 3
The patch was reviewed
04Commenter 1
Combination -
OCCT branch : [archived branch]
Products branch : [archived branch]
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.

Number of compiler warnings:
No new/fixed warnings

Regressions/Differences/Improvements:
No regressions/differences

CPU differences:
Debian80-64:
OCCT
Total CPU difference: 17839.600000000362 / 17844.450000000477 [-0.03%]
Products
Total CPU difference: 11562.480000000109 / 11558.800000000136 [+0.03%]
Windows-64-VC14:
OCCT
Total CPU difference: 19303.28125 / 19431.828125 [-0.66%]
Products
Total CPU difference: 12894.515625 / 12871.203125 [+0.18%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
05Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]

Related records