-
Notifications
You must be signed in to change notification settings - Fork 5
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
Support for non-MRTK projects #3
Comments
|
@xiaomaoo It seems like you are missing the Mixed Reality Toolkit in your project https://github.com/Microsoft/MixedRealityToolkit-Unity As this issue states, it would be a nice feature if this supported non-MRTK projects. Right now, that is not the case. |
Thanks for reply! |
@xiaomaoo very sorry for the late reply. I honestly missed the GitHub notification about your message among other e-mails after I returned from holiday. You are correct that this library should ideally be functional without the MRTK. In fact I intend to remove RefinableModel for now because TwoHand and BoundingBoxRig do not work correctly when you are "inside" the hologram (as is the case when positioning building-scale models). Ray Refinement and Nudge Refinement work far better in this scenario anyway. I did want to mention that we are considering contributing this library into the MRTK. However, we would not plan to do that until MRTK v-Next ships. v-Next has much better support for isolated or "standalone" components which can be used on their own without taking a dependency on the rest of the MRTK. We would love to hear your thoughts on that approach. |
@xiaomaoo I forgot to answer your question about persistence. You're correct that the Persistence sample is the right place to look. Right now that sample serializes the spatial graph into a json string or deserializes a json string into a spatial graph. I think it's good feedback that this sample could be improved. The SA-RefinementBuilding sample does not currently use persistence. The way I would probably implement it is to have a voice command that saves the spatial graph to a text file in the applications isolated storage folder. The next time the application runs I would check to see if that file exists and if so I would load the spatial graph from that file instead of starting with a new spatial graph. I hope that makes sense. I will create a new issue about improving those samples. |
SpatialAlignment would be helpful for other platforms including iOS and Android, but currently components of the library require MRTK. (Specifically RefinableModel uses TwoHand and BoundingBoxRig). The library should be reorganized in some way that allows the MRTK-dependent components to be deselected during library import.
The text was updated successfully, but these errors were encountered: