-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add one-line descriptions to remaining examples #8921
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
Comments
@choldgraf how did you generate this list? Is it easy to re-generate it? |
I believe it was pulled from another PR :-/ I didn't generate it |
Can it be split in multiple PRs (seeing the number of files :D) ? Does it just require a one liner about the example in the end? |
@divyam3897 Yes, splitting into has many PRs as is required is fine. I one sentence description is the minimum, a longer description would be better! I think updates to clarify the examples would also be OK to include. |
@tacaswell I made a PR for the first set of examples starting from event description. Can you review them and suggest changes if any required? :) |
can you link to the PR? :-) |
@choldgraf sure #9333 :) |
@choldgraf @tacaswell second PR to the list #9341 for images-contours_and_fields |
@choldgraf @tacaswell third one #9392 for subplots_axis_and_figures.
|
I'm not sure about number 1. (that link is to old docs though, so maybe try matplotlib.org ?), but for number 2. all the |
Hi, I want to work on this. How do I go about it? Could someone help me with it? |
Hi, I tried doing the fixes. Please do let me know if this works: |
IMHO it's more important to consolidate the number of examples. This holds particular for the "small" examples that don't have descriptions. Adding an one-line description can still be done on a case-by-case basis. I don't think we need an open issue to track this. The above list is outdated anyway. |
Many examples don't have one-line descriptions underneath the title. We should add these in for consistency / readability / sphinx-gallery goodness. Here's a list of files to looks for:
The text was updated successfully, but these errors were encountered: