Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Evaluate new solver system in the context of the Object Manipulator #11459

Closed
srinjoym opened this issue Apr 11, 2023 · 1 comment
Closed

Evaluate new solver system in the context of the Object Manipulator #11459

srinjoym opened this issue Apr 11, 2023 · 1 comment
Assignees

Comments

@srinjoym
Copy link
Contributor

No description provided.

@srinjoym srinjoym self-assigned this Apr 11, 2023
@srinjoym
Copy link
Contributor Author

Investigated the draft PR with the new transformations system and Object Manipulator today: #11323

Basic manipulation with the new ObjectManipulator worked fine (near/far interaction to move).
However, the transformations system still seems to be unfinished:

  1. Bug: I was unable to add any constraints other than the MinMaxScale constraint. (hitting an exception when try to create the class)
  2. Bug: Setting min and max scale to identical causes the object to start stuttering
  3. Bug: Even with the MinMaxScale constraint, I was able to scale the object up beyond the limit. The script would snap the object back to the limit once I stopped the interaction. The constraint system doesn't allow the user to scale the object beyond the limit on the initial interaction
  4. The MinMaxScale transformation doesn't have the "relative to initial state" flag yet which the MRTK 2 defaults to.

Based on these bugs, I would suggest holding off on the transformations system for GA and adding it in a future release.

@AMollis AMollis closed this as completed Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants