[MRTK3] Constraint Manager - Rotation Constraint is missing

microsoft / MixedRealityToolkit-Unity

This repository is for the legacy Mixed Reality Toolkit (MRTK) v2. For the latest version of the MRTK please visit https://github.com/MixedRealityToolkit/MixedRealityToolkit-Unity

https://aka.ms/mrtkdocs

MIT License

6k stars 2.12k forks source link

[MRTK3] Constraint Manager - Rotation Constraint is missing #10762

Closed cre8ivepark closed 1 year ago

cre8ivepark commented 2 years ago

This issue has been migrated a new MRTK repository, and the status of this issue will now be tracked at the following location:

https://github.com/MixedRealityToolkit/MixedRealityToolkit-Unity/issues/288

Describe the bug

In the Constraint Manager options, many constraint options that were available in MRTK2 is missing in MRTK3. For example, Rotation Constraint is missing which is needed to make the object stay vertical while moving/manipulating.

2022-07-18 15_29_33

Zee2 commented 2 years ago

The reason most of the constraints are still missing is because we want to rewrite the constraints/solvers system into a single unified system, a la #10759 . @RogPodge do you think we should add/port the rest of the legacy constraints even though we really don't like the existing constraints/solvers situation?

cre8ivepark commented 2 years ago

The reason most of the constraints are still missing is because we want to rewrite the constraints/solvers system into a single unified system, a la #10759 . @RogPodge do you think we should add/port the rest of the legacy constraints even though we really don't like the existing constraints/solvers situation?

I see! I thought it was an editor UI bug not showing the full list of constraints.

holomatt commented 1 year ago

This issue is currently blocking our project from proceeding to use ObjectManipulator for anchor manipulation - we'd like to use RotationAxisConstraint to prevent roll, but it'd unclear how to add it back in. Similarly, MoveAxisConstraint is fairly essential. I have use cases for pretty much all of the constraints including MaintainApparentSize, but those two are the P0's for us currently.

RogPodge commented 1 year ago

Closed with #11085