| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
New page in the docs: https://output.circle-artifacts.com/output/job/4a00a38f-319f-44c6-bb1b-f12494152fc1/artifacts/0/doc/build/html/api/toolkits/mplot3d/view_angles.html |
Sorry, something went wrong.
|
I think that it is possible to store an svg-file and use that? (There's been a similar case in SymPy reducing the file size from MBs to kb... I do not know if any specific settings are required. But most browsers can render svg anyway?) But the image looks good! |
Sorry, something went wrong.
|
PNG size can be reduced by factors if put through a compressor like https://tinypng.com/. I think if applied to both images, sizes are reasonable small. The real advantage of SVG would be that we could adapt it later on, but I'm ok with PNG if that's what you can generate. Content:
|
Sorry, something went wrong.
|
PNGs are smaller, thanks for the site. The issue with svg is that different browsers render differently, and we actually have a great example right here. In the screenshot of the svg image for @timhoffm it looks like the arrows are not present for some reason, whereas I see arrows colored black in the previews, and when I open in inkscape the arrows colored the same as their lines (what I intended and what the png shows). Tim, do the arrows there resolve the overlaying lines looking weird? Updated to add a little camera cone, I think I like it better even if it makes the plot a bit busier. |
Sorry, something went wrong.
|
Wondering if for this PR or a follow on, the section in the 3D FAQ can be added/consolidated into the doc? |
Sorry, something went wrong.
|
Are you asking for the view angles to be added to the FAQ, or the FAQ info to be interspersed elsewhere where it makes sense? |
Sorry, something went wrong.
|
If the relevant parts of the FAQ can be incorporated into the new document so that information isn't spread across two. |
Sorry, something went wrong.
|
I don't see anything in the FAQ that I think it makes sense to pull out to the new page, but I'm happy to just put all the new info into the existing FAQ. |
Sorry, something went wrong.
|
Then just forget my suggestion, (sorry) since I like this is a standalone & think it's more discoverable that way (I'm not a big fan of FAQs) |
Sorry, something went wrong.
Sorry, something went wrong.
|
The camera looks good and the arrow also resolves my line issue 👍. One additional idea for decluttering: How about only indicating the roll direction: Technically, that's a bit different than the azimuth/elevation arrows, which show magnitude, but I think people still understand what roll is. Maybe even show the viewing plane for roll=0, which again may be easier to comprehend as the user has less angles to process. The roll arrow then just indicates "this is the direction roll would apply to" |
Sorry, something went wrong.
|
OK! I get the size and rendering issue, but wouldn't it make sense to have the "box" as svg so that it can easily be resized when printing? (Given that the size is not completely off. Or generate that image on the fly? I assume it is generate with mpl?) |
Sorry, something went wrong.
|
Ok, addressed those comments. I didn't totally go with Tim's suggestion since I think showing the roll angle zero reference is important, but I did take out two of the other lines there that weren't adding much. And to Oscar's suggestion, I made the unfolded box plot a gallery example with the generating code right there. I think it all looks good! Built docs API page: https://output.circle-artifacts.com/output/job/0162ce41-cee3-47e2-a5a8-61aa17145914/artifacts/0/doc/build/html/api/toolkits/mplot3d/view_angles.html |
Sorry, something went wrong.
There was a problem hiding this comment.
Looks great!
Optional: It may be neat to add the fold lines to the unfolded view. But we can always add that later. I'd like to get this into 3.6 docs. So we should merge soon.
Sorry, something went wrong.
| axd[plane].view_init(elev=views[i][1][0], | ||
| azim=views[i][1][1], | ||
| roll=views[i][1][2]) | ||
| axd[plane].set_box_aspect(None, zoom=1.25) |
There was a problem hiding this comment.
Would this be a good use for axs[plane].axis('equal')? 😄
Sorry, something went wrong.
There was a problem hiding this comment.
A cutout should work for any aspect box :). But right now that command would only change the data limits! Need #23552 to merge in to change the box.
Sorry, something went wrong.
There was a problem hiding this comment.
Ok, let's to this later then. Since we've already released 3.6 RC1, #23552 can't go into 3.6 anymore.
Sorry, something went wrong.
…721-on-v3.6.x Backport PR #23721 on branch v3.6.x (3d plot view angle documentation)
| Back | FazBrowse Home | New Git URL |
PR Summary
Adds documentation and images for the 3d plot view angle conventions. Also adds an example showing the primary view planes. Closes #21508
Inkscape file to generate the view angles plot is attached (the arrows render weird in the svg preview here for some reason).

PR Checklist
Tests and Styling
Documentation