-
Notifications
You must be signed in to change notification settings - Fork 679
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
Fix points raycast #1342
Fix points raycast #1342
Conversation
There are two issues: 1. `_position` is the the world position. In the original code, _position is retrieved from the bufferattribute which is local position 2. the matrix that is applied in the original code is the parent matrixWorld. This is also in line with the line 58.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit bfc762d:
|
Would be more correct since we're using the class' local transforms. The difference between the two lines is whether the ray should do the same or exist as a sibling of the same parent.
@promontis is this done? |
This version is better than the current version, so you might want to merge it in. I'm currently running in a couple of problems with this code, namely:
|
🎉 This PR is included in version 9.66.5 🎉 The release is available on: Your semantic-release bot 📦🚀 |
There are two issues:
_position
is the world position. In the original code, _position is retrieved from the bufferattribute which is local positionthe matrix that is applied in the original code is the parent matrixWorld. This is also in line with the line 58.