Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

DOC / BUG: Better example for 3D axlim_clip argument #28984

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

Merged
QuLogic merged 2 commits into matplotlib:main from scottshambaugh:axlim_clip_example
Oct 17, 2024

Conversation

Copy link
Contributor

@scottshambaugh scottshambaugh commented Oct 16, 2024
edited
Loading

PR summary

I don't like the example I made that shows clipping to 3D axis limits in #27349. It's not clear which of the random points are actually supposed to be outside the axis limits. Switched the gallery example and what's new plot to a continuous function, which I think is easier to interpret.

Before:
image

After:
sphx_glr_axlim_clip_001_2_00x

PR checklist

Copy link
Contributor Author

Update: this also uncovered a bug in axlim_clip for Line3DCollection, which is now fixed.

@scottshambaugh scottshambaugh changed the title (削除) DOC: Better example for 3D axlim_clip argument (削除ここまで) (追記) DOC / BUG: Better example for 3D axlim_clip argument (追記ここまで) Oct 16, 2024
@scottshambaugh scottshambaugh force-pushed the axlim_clip_example branch 2 times, most recently from 8d1bd4e to 87bad30 Compare October 16, 2024 16:46
@scottshambaugh scottshambaugh marked this pull request as ready for review October 16, 2024 16:46
scottshambaugh and others added 2 commits October 16, 2024 21:27
Tests
Tests
Tests
Update lib/mpl_toolkits/mplot3d/art3d.py
Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
Copy link
Contributor Author

Updated to add tags to the example

@QuLogic QuLogic merged commit 940b897 into matplotlib:main Oct 17, 2024
41 of 43 checks passed
@scottshambaugh scottshambaugh deleted the axlim_clip_example branch October 17, 2024 03:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@QuLogic QuLogic QuLogic approved these changes

@ksunden ksunden ksunden approved these changes

Assignees
No one assigned
Labels
Documentation: examples files in galleries/examples PR: bugfix Pull requests that fix identified bugs topic: mplot3d
Projects
None yet
Milestone
v3.10.0
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /