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

scipy.spatial.transform.rotation documentation #12742

Open
valschmidt opened this issue Aug 19, 2020 · 2 comments
Open

scipy.spatial.transform.rotation documentation #12742

valschmidt opened this issue Aug 19, 2020 · 2 comments
Labels
Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org scipy.spatial

Comments

@valschmidt
Copy link

I would like to see the documentation that better explains the "angles" argument to functions like from_euler() and elsewhere in the Rotation class. It does not seem to be explicitly stated anywhere, for example, that the angles are [rotation_about_z, rotation_about_y, rotation_about_x]. And while one can specify the order of rotations with the seq argument, without that information one has to reverse engineer which angles apply to which rotation.

Thanks!
-Val

@ilayn ilayn added Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org scipy.spatial labels Aug 19, 2020
@ilayn
Copy link
Member

ilayn commented Aug 19, 2020

Hi @valschmidt Care to submit a PR to make the documentation clearer? It is sometimes difficult for us to come up with better explanation than what we have but reviewing is epsilon-more easier for us to get the context.

@valschmidt
Copy link
Author

Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org scipy.spatial
Projects
None yet
Development

No branches or pull requests

2 participants