-
Notifications
You must be signed in to change notification settings - Fork 209
Description
Couple of complaints about the spatialcontroller package, in the AccessoryTrackingHelper class
line 7
using Unity.VisualScripting;
this shouldn't be here - it's not used, but causes compile errors if you don't have that package. This seems like it's just clearly a bug/mistake
line 14
using Unity.PolySpatial;
We don't use polyspatial at all - polyspatial is added as a dependency to the package just to check if you are in a specific camera mode. It's so much extra bloat just for something we don't care about or need. I believe you can use "Version Defines" to make this code optional for people who have polyspatial in their project, rather than making it a forced dependency
Also, when using the package, it seems to require a hand tracking description - even though we aren't really using hand tracking. To make things slightly more annoying, the hand tracking description box in the xr project settings is greyed out if you don't have unity.xr.hands in your project