Thanks for the PR! I unfortunately don't have concrete suggestions but I'm wondering if some of these examples can be smaller/more straightforward? The goal of this gallery isn't necessarily pretty pictures, just examples that clearly show what data goes into what plots & what plots are available.

Thanks for the feedback. I have curated a more straightforward set of examples in matplotlib/plot_types/3D.

Sorry about all of the commits, this is my first pull request and I'm still getting used to Git. If I need to rebase or do anything else, please let me know. I will wait for feedback before making any new commits or merges.

It looks like your code got muddled in w/ the rebase.

Git is a pain. I'm gonna suggestcherry picking b/c it's what I just did to clean a branch, but there may be an easier way to clean your branch. Also feel free to drop by our incubator channel w/ questions!

1. locally checkout main
   git checkout main
2. rebase main to upstream main
   (on main)git rebase upstream/main
3. checkout new branch FIX (while main is checked out, so FIX is branched off a synced main)
   (on main)git checkout -B FIX
4. on new branch FIX, cherry pick just the commits that are relevant to this PR
   (on FIX)git cherry-pick 910ed53147058d02f43bd02865e7420be08e4969 53ea816a5e3e0a3002026c059f69aaf53c284e4b
5. delete your local 'add-3d-plots-to-gallery'
   (on FIX)git branch --delete add-3d-plots-to-gallery
6. rename FIX to 'add-3d-plots-to-gallery'
   (on FIX)git branch -m add-3d-plots-to-gallery
7. checkgit log to check that the only commits on this branch are the ones relevant to this pr. Here's a simplified variant/the numbers may be different for you)

commit 910ed53147058d02f43bd02865e7420be08e4969 (HEAD ->'add-3d-plots-to-gallery, origin/'add-3d-plots-to-gallery)3d example codecommit cada8fb565b1e1cfeb076f1f6ab791b1640d387d (upstream/main, main)Merge: 363713442b da3790fc4e

8. push back to this branch

These examples need to become more in line with the rest of plot types (take the others as example):

• Adapt the styling
• Make the text minimal

Almost there! The gallery examples follow a# make data, # plot format which you've done in trisurf, voxels; please amend scatter, surface, and wireframe to match.

Also, the failed doc tests means I can't see how the new style is being applied to the gallery entries. I think sphinx is mad b/c you're using the same names as the gallery example:

WARNING: Duplicate example file name(s) found. Having duplicate file names will break some links. List of files: ['../plot_types/3D/scatter3d.py', '../plot_types/3D/surface3d.py', '../plot_types/3D/trisurf3d.py', '../plot_types/3D/voxels.py', '../plot_types/3D/wire3d.py']

so please change the names - appending simple should be fine.

Thanks again for feedback, everyone.

Where was that warning message about duplicated file names? I didn't see it under "Details" for "View the built docs" or "ci/circleci: docs-python38", but my browser was having issues showing the previous circleci.com page.

Edit: I see the warnings on circleci now.

This is looking good. You have a build failure however.

1. Please remove the tick labels
2. In the description, make it clear that these are from a toolkit. Something like "3D plots using 3D mpltoolkit". They are a bit confusing because they are not in the main library.

Thanks!

One way to remove the tick labels is by addingax.set_xticklabels([]),ax.set_yticklabels([]), andax.set_zticklabels([]).

Is this acceptable? Or is there a cleaner, preferred way to remove all tick labels? Maybe through styling?

the examples all useax.set to set stuff like ticks and you may need to edit the styles athttps://github.com/matplotlib/matplotlib/tree/main/lib/matplotlib/mpl-data/stylelib to add in 3D configs

For the surfaces, I agree that a color gradient could increase visual clarity. I think these changes work well while adding minimal complexity.

Regarding the voxels, the color-related code was removed from theoriginal gallery example. Is reintroducing distinct colors worth increasing the Plot Types code complexity?

For the wireframe, after reading through the docs and doing some testing, it is unclear if a z-coordinate-dependent color gradient can be added to each plotted line. And the cmap keyword, e.g.cmap=cm.viridis, does not change the wireframe appearance.

Yup you're right, wireframe doesn't support colormapping. Sorry about that! And I don't have strong opinions on voxels, so will agree w/ you to keep it on the simple less pretty side.

Oh this is so pretty and I'm just giving@jklymak and@timhoffm a day or two to comment before I merge.

Congrats 🥳 on your first PR into matplotlib, thank you so much for your patience with our feedback 😅, and we hope to see you again!