-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
DOC: Add a plot to margins() to visualize the effect #28808
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
Conversation
This seems like a complicated example for the ApI docs? Would we not prefer just a link-out here to a standalone example? |
This is not an example in the sense that the code is relevant. The rendered picture is. The code does not show up in the rendered docs: we could alternatively embed a static image, but it's slightly simpler to adapt/maintain from code, and it's and "added feature" that an interested user can get the source via link. If you're concerned about raw docs, we could put the code into a separate file and
|
I think that's way clearer as an explaination! What about putting it in the gallery with a That way it's available for quick lookup for folks browsing the gallery and who may want the code, but also the tag indicates that this isn't intended as teaching code? |
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.
While I think it'd be cool as an example, I also don't kinda like us just having a folder for these sorta plots/might help in gallery cleanup in a different way.
This is probably fine, though a link would work for rendered and raw docs. However the docs are not building on this pr |
06bba7e
to
bf267ad
Compare
Links sound better than they are for raw docs. A link would look like Another issue with putting this into the gallery as is, is that the plot is specifically designed for the visual result, it's not an instructive code example for margins. Generally, I believe, there's a place for "plots embedded into the docstrings". Visualizations help. Naturally, we can only do them in HTML, not in raw docs). Putting the plot code into separate files helps to keep the raw docstings readable. Note that we don't have to go with separate files in .rst docs. They are not read in raw by users, so inline This PR specifically, makes |
bf267ad
to
32d9450
Compare
32d9450
to
8ea86fe
Compare
I really need to proofread what I write and am very sorry for the confusion! I don't know how the don't got in there - I meant to say that I really like the folder for collecting these kinds of images cause I agree we need more of em. 🤦♀️ I just didn't merge in case you liked the gallery idea. . |
No description provided.