Skip to content
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

Extra vector angle function. #407

Closed
wilham7 opened this issue Jan 26, 2023 · 1 comment · Fixed by #413
Closed

Extra vector angle function. #407

wilham7 opened this issue Jan 26, 2023 · 1 comment · Fixed by #413
Assignees
Labels
enhancement 📢 Issues describing an enhancement or pull requests adding an enhancement. minor Pull requests requiring a minor version update according to semantic versioning.

Comments

@wilham7
Copy link

wilham7 commented Jan 26, 2023

Hey there,

I've been using the VectorAngle and VectorAngleOnPlane functions.

I would like the possibility to use the VectorAngleOnPlane function and ALWAYS receive the angle between A and B, regardless of if it is reflex or regular angle. This is how the VectorAngle function works as does the Rhino vector and angle method.

https://developer.rhino3d.com/api/RhinoCommon/html/M_Rhino_Geometry_Vector3d_VectorAngle_1.htm

Thanks!

@sonomirco
Copy link
Collaborator

Using the current VectorAngleOnPlane, the angle returned could be internal or the reflexAngle.
It would be nice to have this method passing returning only the angle between the vectors as Vector3d.VectorAngle in Rhinocommon does.
mcneel/rhinocommon#178

@sonomirco sonomirco self-assigned this Jan 29, 2023
@sonomirco sonomirco added enhancement 📢 Issues describing an enhancement or pull requests adding an enhancement. minor Pull requests requiring a minor version update according to semantic versioning. labels Jan 29, 2023
@sonomirco sonomirco mentioned this issue May 4, 2023
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 📢 Issues describing an enhancement or pull requests adding an enhancement. minor Pull requests requiring a minor version update according to semantic versioning.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants