-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
DOC: Add more 3D plot types #28243
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: Add more 3D plot types #28243
Conversation
174ed1e
to
014e434
Compare
@@ -0,0 +1,27 @@ | |||
""" | |||
================ | |||
plot(xs, ys, zs) |
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.
plot(xs, ys, zs) | |
plot(x, y, z) |
For consistency. Best also change the variable names below.
oh, I see, the existing scatter() was using this convention, but that's really the odd ball and should be changed as well.
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.
The Axes3D.plot
function has xs
, ys
, and zs
as the argument names, I figure it's best to be consistent with that?
https://matplotlib.org/stable/api/_as_gen/mpl_toolkits.mplot3d.axes3d.Axes3D.plot.html#mpl_toolkits.mplot3d.axes3d.Axes3D.plot
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.
Add it to the list of annoying API inconsistencies
…243-on-v3.9.x Backport PR #28243 on branch v3.9.x (DOC: Add more 3D plot types)
PR summary
We were missing a good number of 3D plot types on this page: https://matplotlib.org/stable/plot_types/
I added examples for
plot
,quiver
,bar3d
, andstem
.PR checklist