search

XRTK / com.xrtk.core

The Official Mixed Reality Framework for Unity
https://xrtk.io
MIT License
309 stars 34 forks source link

Port updated Solvers functionality from MRTK #260

Closed SimonDarksideJ closed 2 years ago

SimonDarksideJ commented 5 years ago

XRTK - Mixed Reality Toolkit Feature Request

Is your feature request related to a problem? Please describe

MRTK recently updated Solver logic / code, to keep in line we should port / update the Solver logic

https://github.com/microsoft/MixedRealityToolkit-Unity/pull/5416

Ideally, this area still needs heavy rework to make it work more in the scriptableobject configuration approach, but for now, it should keep pace

How would you classify your suggestion

hridpath commented 5 years ago

This would be good, sooner than later. @StephenHodgson 😺

StephenHodgson commented 5 years ago

I got a great idea for this.

I'd like to add a IMixedRealitySover interface with a lot of the public members from the established classes.

Then have a solver system that works a lot like our service locator, that registers the solvers, giving them a priority and then sorting them, then calling update in a specific order.

I think the tricky bits are going to be the protected members that are utilized a lot for the established functionality.

SimonDarksideJ commented 5 years ago

I love this idea, I just want to clear some time to focus on solving this issue 😂