FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

DOC: clarify explanation of axline in infinite lines example by amishamehta99 · Pull Request #31316 · matplotlib/matplotlib · GitHub

  1. matplotlib
  2. matplotlib

Repository navigation

DOC: clarify explanation of axline in infinite lines example - #31316

Merged
timhoffm merged 1 commit into
matplotlib:mainfrom
amishamehta99:improve-axline-doc
Mar 17, 2026
ConversationCommits (1)ChecksFiles changed
Merged

DOC: clarify explanation of axline in infinite lines example#31316
timhoffm merged 1 commit into
matplotlib:mainfrom
amishamehta99:improve-axline-doc

Conversation

amishamehta99 commented Mar 16, 2026

Copy link
Copy Markdown
Contributor

Improves the explanation of axline in the Infinite Lines gallery example.
The previous description did not explain how the line is defined.
This update clarifies that axline can be defined by two points or by a point and a slope.

github-actions Bot added the Documentation: examples files in galleries/examples label Mar 16, 2026

amishamehta99 commented Mar 16, 2026

Copy link
Copy Markdown
Contributor Author

Hi! I’ve made a small documentation improvement to clarify the explanation of axline in the Infinite Lines example.
Could someone please review it when convenient? I’m happy to make any changes if needed. Thanks!

rcomer commented Mar 16, 2026

Copy link
Copy Markdown
Member

@amishamehta99 this includes commits from your other PR. Please remove them.

timhoffm reviewed Mar 16, 2026
View reviewed changes

timhoffm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Wording is good, but you’ve accidentally added the sidebar/PDF changes. Please remove them.

amishamehta99 commented Mar 16, 2026

Copy link
Copy Markdown
Contributor Author

Wording is good, but you’ve accidentally added the sidebar/PDF changes. Please remove them.

Thanks for pointing that out! I accidentally included commits from my other PR. I’ll clean up the branch and update this PR to include only the axline documentation change.

amishamehta99 commented Mar 16, 2026

Copy link
Copy Markdown
Contributor Author

@amishamehta99 this includes commits from your other PR. Please remove them.

Thanks for pointing that out! I've removed the unintended changes and pushed an updated version of the branch.

timhoffm added this to the v3.11.0 milestone Mar 17, 2026
timhoffm merged commit 40730d1 into matplotlib:main Mar 17, 2026
20 checks passed

timhoffm commented Mar 17, 2026

Copy link
Copy Markdown
Member

Thanks @amishamehta99

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

timhoffm timhoffm approved these changes

Assignees

No one assigned

Labels

Documentation: examples files in galleries/examples

Projects

None yet

Milestone

v3.11.0

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL