-
Notifications
You must be signed in to change notification settings - Fork 36
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
WARN THREE.Quaternion: .inverse() has been renamed to invert(). #28
Comments
same issue here |
FYI, Looks like a fix was merged here but i'm still getting the error as well. I've asked the maintainer why the error persist even after the MR was merged. |
I try to modify OrbitControls.js which is belong to the expo-three-orbit-controls. Go to line 634, and I replaced inverse() to invert(). Then It works! |
Can we get this change in the package? Is this project abandoned? |
for the moment we can fork this repo and do that change manually. |
You can check line 634 current release on NPM do not have the change https://www.npmjs.com/package/expo-three-orbit-controls?activeTab=code |
After making this change, I can't zoom in or zoom out, the object disappears. |
modify Quaternion's prototype can fix this |
I am using expo-three with three.js.
After updating three.js to 0.128 I get the warning:
WARN THREE.Quaternion: .inverse() has been renamed to invert().
But I have no line in my code with: .inverse()
OrbitControls uses e.g. the
var quatInverse = quat.clone().inverse();
Yes, without OrbitControlsView there is no warning!
The text was updated successfully, but these errors were encountered: