Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC: Standardize example titles - part 2 #28641

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
merged 1 commit into from
Aug 2, 2024

Conversation

timhoffm
Copy link
Member

@timhoffm timhoffm commented Aug 1, 2024

Follow up to #28544. As there:

Following recommendations from #28527, this improves example titles. Take this as an incremental improvement. I've changed what I saw at a glance when going through the examples once. Certainly, one could do further improvements, but that can be done in follow-ups.

@github-actions github-actions bot added the Documentation: examples files in galleries/examples label Aug 1, 2024
Fill Between with transparency
==============================
==================================
``fill_between`` with transparency
Copy link
Member Author

@timhoffm timhoffm Aug 1, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So far, there's only one case of formatting example titles (plt.subplots). I feel it's slightly better to format function names as literal rather than just have them as plain text. But no strong opinion. We can also leave out the formatting if desired.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If sphinx doesn't complain, I fully support formatting function names this way

@timhoffm timhoffm force-pushed the doc-example-titles branch 3 times, most recently from af9f872 to c6cf03f Compare August 1, 2024 22:47
Copy link
Member

@story645 story645 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for putting in easy to review on phone PRs 😅

Filling the area between lines
==============================
===========================
Fill the area between lines
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Fill the area between lines
Fill area between horizontal curves

Mostly to draw distinction between fill_betweenx,

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've kept the title (only adding "two"): "Fill the area between two lines"

While the docstring says "horizontal curves" and I've changed fill_betweenx to "Fill the area between vertical curves", I like the simplicity of "lines". This alludes to Line2D. The term "horizontal curve" insinuates precision, but is acutally still vague.

What "horizontal curve" actually means, are two curves that are defined at the same horizontal positions x. Visually, the lines can be quite non-horizontal. But I'm unable to express this in the title, so I stick to the very generic description, which I believe will catch people intuitively.

Adding lines to figures
=======================
=====================
Add lines to a figure
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Add lines to a figure
Draw lines without an axes

The "no axes" part isn't implied by the title and is I think the important part.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The point is really "to a figure" not "without an axes".

Getting a bit out of scope, but small enough to not make a separate PR: I've changed the content of the example to something more practical:

image

Embedding in GTK3 with a navigation toolbar
===========================================
=======================================
Embed in GTK3 with a navigation toolbar
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Embed in GTK3 with a navigation toolbar
Embed plot with a navigation toolbar in GTK3

I think here it might be better to have the subject

Embedding in GTK4 with a navigation toolbar
===========================================
=======================================
Embed in GTK4 with a navigation toolbar
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as 3, also do we want this as one example (embed in GTK) and then maybe tab GTK3 and GTK4 or have them as subheadings?

Embedding in GTK4
=================
=============
Embed in GTK4
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same, do we want one GTK example and then the diff versions as subheadings?

Embedding in wx #2
==================
==============
Embed in wx #2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why wx #2 when it's wxagg in text?

Embedding in wx #5
==================
==============
Embed in wx #5
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can all the wx be combined?

@timhoffm timhoffm closed this Aug 2, 2024
@timhoffm timhoffm deleted the doc-example-titles branch August 2, 2024 07:38
@timhoffm timhoffm restored the doc-example-titles branch August 2, 2024 07:41
@timhoffm timhoffm reopened this Aug 2, 2024
@timhoffm timhoffm force-pushed the doc-example-titles branch from c6cf03f to 5cf1437 Compare August 2, 2024 08:01
@timhoffm
Copy link
Member Author

timhoffm commented Aug 2, 2024

I leave all the embed stuff for now. That should be resturctured in a separate PR.

Copy link
Member

@story645 story645 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

slight grammar consistency nits, can self merge after fixing those.

Fill Betweenx Demo
==================
=========================================
Fill the area between two vertical curves
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Fill the area between two vertical curves
Fill the area between two vertical lines

since you like lines better, since the terminology should stay consistent across both. I'm not sure if I think this example should be combined w/ the other one.

Adding lines to figures
=======================
=================================
Add lines to directly to a figure
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Add lines to directly to a figure
Add lines directly to a figure

grammar nit,

and my qualm/concern is folks not understanding what "to a figure" means, but the more I unpack that the more I'm thinking it probably can't be solved in a title

Follow up to matplotlib#28544. As there:

Following recommendations from matplotlib#28527, this improves example titles.
Take this as an incremental improvement. I've changed what I saw  t a
glance when going through the examples once. Certainly, one could do
further improvements, but that can be done in follow-ups.

Co-authored-by: hannah <[email protected]>
@timhoffm timhoffm force-pushed the doc-example-titles branch from 5cf1437 to a4e9e07 Compare August 2, 2024 20:30
@timhoffm timhoffm added this to the v3.10.0 milestone Aug 2, 2024
@timhoffm timhoffm merged commit c17197c into matplotlib:main Aug 2, 2024
22 checks passed
@timhoffm timhoffm deleted the doc-example-titles branch August 2, 2024 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation: examples files in galleries/examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants