MixedRealityToolkit / MixedRealityToolkit-Unity

This repository holds the third generation of the Mixed Reality Toolkit for Unity. The latest version of the MRTK can be found here.
BSD 3-Clause "New" or "Revised" License
374 stars 96 forks source link

Solver Handler not switching hands when tracking both left and right Interactors #325

Closed IssueSyncBot closed 1 year ago

IssueSyncBot commented 1 year ago

Original issue opened by:

@srinjoym srinjoym


Bug found during investigation of validating surface magnetism solver.


ISSUE MIGRATION

Issue migrated from: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/11434

IssueSyncBot commented 1 year ago

Original comment by:

@srinjoym srinjoym


Validated on MRTK 3 - the core solver seems to be working as expected. The only issue was in the Solver Handler script - if it is set to track both hands through two interactors, then it will always prefer the left hand even if it's not currently tracked. This probably needs to be fixed as I would expect the solver to start tracking the right hand if the left goes out of view

Here's a video of the current state:

https://user-images.githubusercontent.com/7866060/227638284-24280b33-8d0f-4857-8e12-7f291794756e.mp4

IssueSyncBot commented 1 year ago

Original comment by:

@AMollis AMollis


@srinjoym , this seems like a duplicate of this #11398 can you please confirm?

IssueSyncBot commented 1 year ago

Original comment by:

@AMollis AMollis


Removing "blocking" until we verify there is no workaround for this issue. Suspected workaround:

IssueSyncBot commented 1 year ago

Original comment by:

@srinjoym srinjoym


@AMollis This workaround doesn't work for me. Even with two solver handlers (one for left one for right) the object only follows the left hand. Suggests this is a separate problem from the other issue

Update: Looked into this further and the difference between the two issues is that I'm observing the issue with two interactors (far rays) vs. Matt's issue is observing the issue tracking two hand joints. I think we can make a fix for this specific issue without the upgrade to XRI 2.3 and I'm drafting a PR right now