-
Notifications
You must be signed in to change notification settings - Fork 441
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 random behavior of CylinderStructured #442
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the casting it looks good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good (except for a nit and what @prisae wrote). However does it actually resolve #439? They ask for being able to specify a second direction for the seed point. This PR changes that direction from a random one to the projection of the z-axis to the plane vertical to the cylinder axis. That might be enough for their purposes, but is not exactly the same thing.
Co-Authored-By: Simon Frei <freisim93@gmail.com>
Resolve #439
This should fix the random location of the seed point by not arbitrarily rotating the mesh to the given direction vector.
Someone may need to check my vector math