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

[Doc]: Create diagram to show rotation directions for 3D plots #21508

Closed
scottshambaugh opened this issue Oct 30, 2021 · 3 comments · Fixed by #23721
Closed

[Doc]: Create diagram to show rotation directions for 3D plots #21508

scottshambaugh opened this issue Oct 30, 2021 · 3 comments · Fixed by #23721
Milestone

Comments

@scottshambaugh
Copy link
Contributor

scottshambaugh commented Oct 30, 2021

Documentation Link

No response

Problem

PR #21426 adds a "roll" angle to viewing 3D plots for v3.6.0. We need a picture like the one below (which comes from here) to communicate to users what the viewing angles are.

Compared to the picture below, we call that angle "roll" rather than twist. Note also the arrows show the movement of the axes, not the camera. This is the same rotation convention that pyvista uses.

We should probably also have an image showing the movement of the camera, not the axes.

image

This isn't my strong suit so I'll leave it for someone else to complete.

Suggested improvement

No response

Matplotlib Version

3.5.0.dev2389+g8ad61bdacd

Matplotlib documentation version

3.4.3

@scottshambaugh
Copy link
Contributor Author

scottshambaugh commented Jul 22, 2022

As a bonus, would be good to show the u, v, w view axes as added in #23449

They're defined like the axes in this image from this page, where x, y, z in this image map directly to u, v, w.
image

@scottshambaugh
Copy link
Contributor Author

scottshambaugh commented Aug 18, 2022

As another bonus, the "unfolded" 3D plot from here would be good to include. This all should probably be a full standalone page of documentation.

image

@scottshambaugh
Copy link
Contributor Author

scottshambaugh commented Aug 23, 2022

Took at stab at this today:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants