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

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

Open IssueSyncBot opened 1 year ago

IssueSyncBot commented 1 year ago

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: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/10759

IssueSyncBot commented 1 year ago

Original comment by:

@keveleigh keveleigh


Previously discussed in https://github.com/microsoft/MixedRealityToolkit-Unity/issues/2798 and https://github.com/microsoft/MixedRealityToolkit-Unity/issues/4675 as well.

IssueSyncBot commented 1 year ago

Original comment by:

@SimonDarksideJ SimonDarksideJ


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

IssueSyncBot commented 1 year ago

Original comment by:

@AMollis AMollis


Related PR #11323