OCCT3D
DiscussionsIssue archiveOCCT:Visualization

Archived issue #0030672

Visualization - AIS_InteractiveContext::SetLocation() sets invalid selection BVH flag to removed object

Open CASCADEOCCT:Visualizationclosed5 public notes

Search issues

Description

Calling AIS_InteractiveContext::SetLocation() on object removed from context leads to broken selection after re-displaying it.

This is because SelectMgr_TBU_Remove is overridden by SelectMgr_TBU_None flag.

Steps to reproduce

pload MODELING VISUALIZATION
box b 1 2 3
vclear
vinit View1
vdisplay -dispMode 1 b
vfit
vremove -context b
vlocation b -setLocation 0 0 0
vdisplay -dispMode 1 b



pload MODELING VISUALIZATION
box b 1 2 3
vclear
vinit View1
vdisplay -dispMode 1 -highMode 1 b
vfit
vremove -context b
vlocation b -setLocation 2 0 0
vdisplay -dispMode 1 b
vmoveto 200 200
if { [vreadpixel 150 150 -rgb -name] != "BLACK" } { puts "Error" }
if { [vreadpixel 300 300 -rgb -name] != "DARKGOLDENROD3" } { puts "Error" }
vmoveto 300 300
if { [vreadpixel 300 300 -rgb -name] != "DARKTURQUOISE" } { puts "Error" }

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: Wed Apr 24 09:16:19 2019 +0300

    0030672: Visualization - AIS_InteractiveContext::SetLocation() sets invalid selection BVH flag to removed object
    
    SelectMgr_SelectionManager::Remove() now deactivates selection before removal.
02Author
Patch is ready for review.

http://vm-jenkins-test-12.nnov.opencascade.com:8080/view/CR30672-master-KGV/
03Commenter 3
Branch CR30672 reviewed without remarks.
04Commenter 1
Combination -
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
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: 16569.950000000037 / 16536.609999999906 [+0.20%]
Products
Total CPU difference: 10519.070000000038 / 10546.95000000005 [-0.26%]
Windows-64-VC14:
OCCT
Total CPU difference: 17925.484375 / 17960.6875 [-0.20%]
Products
Total CPU difference: 12157.890625 / 12065.3125 [+0.77%]


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