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.

Sign up for GitHub

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

Jump to bottom

Refactor Solver System to be more extendible and user friendly #289

Open
IssueSyncBot opened this issue Aug 21, 2023 · 3 comments
Open

Refactor Solver System to be more extendible and user friendly #289

IssueSyncBot opened this issue Aug 21, 2023 · 3 comments
Labels
Type: Feature Request A request for a new feature that can be included with the next minor version release.

Comments

@IssueSyncBot
Copy link

IssueSyncBot commented Aug 21, 2023 •
edited
Loading

Original issue opened by:

@RogPodge @RogPodge

Based on comments left from #10748

Basically, Solvers needlessly inherit from Monobehaviors so that they can exist as components on a gameobject, even though they do not perform any independent actions during update. This results in several design quirks, like relying on inspector ordering and having to register and unregister from a SolverHandler.

This issue tracks re-architecting the system to improve on the current design.

ISSUE MIGRATION

Issue migrated from: microsoft/MixedRealityToolkit-Unity#10759
@IssueSyncBot IssueSyncBot added MRTK3 Type: Feature Request A request for a new feature that can be included with the next minor version release. labels Aug 21, 2023
@IssueSyncBot
Copy link
Author

Original comment by:

@keveleigh keveleigh

Previously discussed in microsoft/MixedRealityToolkit-Unity#2798 and microsoft/MixedRealityToolkit-Unity#4675 as well.

@IssueSyncBot
Copy link
Author

Original comment by:

@SimonDarksideJ SimonDarksideJ

Ahh, this brings back such memories @keveleigh and for beating me to referencing that age old request :D

@IssueSyncBot
Copy link
Author

Original comment by:

@AMollis AMollis

Related PR #11323

@AMollis AMollis removed Epic labels Aug 21, 2023
@IssueSyncBot IssueSyncBot mentioned this issue Aug 24, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Feature Request A request for a new feature that can be included with the next minor version release.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@AMollis @IssueSyncBot