DiscussionsIssue archiveOCCT:Visualization

Archived issue #0032750

Visualization, AIS_Manipulator - selection of moved object is broken

Open CASCADEOCCT:Visualizationverified13 public notes

Search issues

Description

Attached objects are moved using SetLocationTransformation() method, and selection of moved object doesn't work.

Steps to reproduce

- Attach manipulator to object.
- Move it half-size right.
- Detach manipulator from object.
- Try moving cursor above left half object.
Result: OK, object is highlighted.
- Try moving cursor above right half object.
Result: KO, object is not detected.

Public activity

13 archived notes

Participants are labeled by their role within this record.

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

[revision removed]


Detailed log of new commits:

Author: drochalo
Date: Mon Nov 20 10:26:51 2023 +0000

    Merge branch 'master' of git.dev.opencascade.org:occt into CR32750

Author: drochalo
Date: Mon Nov 20 10:25:09 2023 +0000

    0032750: Visualization, AIS_Manipulator - selection of moved object is broken
    
    Completed the stop transform action when dragging manipulator with mouse.
    Added context redisplay for update of interactive object sensitive areas.
    Added test.
02Commenter 2
Dear Marina, please review.
Tests in jenkins that show differences are not connected to the changes made.
http://jenkins-test-10.nnov.opencascade.com/view/CR32750-master-drochalo/view/COMPARE/
03Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
04Commenter 3
vfit

I think adding something like:
vfront
vfit
vzoom 0.5

would produce more reliable and handful test.

vsetdispmode 1

Setting "unlit" shading mode would exclude impact of illumination.

#NOTE: issue not reproducible by draw command
#but similar action with mouse would cause error


"Error" therm is confusing in this case. Please clarify that it is not "error" but this particular issue.

if { [vreadpixel {*}$mouse_pick2 rgb name] != "CYAN" } { puts "Error: failed to select object" }

I don't think pixels hunting is reliable enough.
It is better just to produce tree images: unselected, selected from left, selected from right.
05Commenter 2
Branch [archived branch] has been updated by Participant.

[revision removed]


Detailed log of new commits:

Author: drochalo
Date: Tue Jan 23 17:35:56 2024 +0000

    0032750: Visualization, AIS_Manipulator - selection of moved object is broken
    
    Modified test.
06Commenter 6
Dear Ilya,

the modifications were made on the tests. Please review.
07Commenter 3
The modifications are okay. Please provide test results.
08Commenter 8
In annex there are 2 images that followed the same test case (mentioned in the steps to reproduce):
- move_manipulator_KO: code without changes;
- move_manipulator_OK: code with new changes.
Attachment 2 (PNG) (22,099 bytes)   
Attachment 1 (PNG) (23,798 bytes)   
09Commenter 4
Please describe branches to integrate
10Commenter 5
Needs to update branch
11Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
12Commenter 12
Dear Commenter 1,
I have updated the branch, squashed commits and pushed.
The branch to integrate is CR32750.
13Commenter 2
Branch [archived branch] has been deleted by Commenter 5.

[revision removed]