Skip to content

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

Merged
timhoffm merged 1 commit intomatplotlib:mainfrom
amishamehta99:improve-axline-doc
Mar 17, 2026
Merged

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

Conversation

@amishamehta99
Copy link
Copy Markdown
Contributor

@amishamehta99 amishamehta99 commented Mar 16, 2026

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 github-actions bot added the Documentation: examples files in galleries/examples label Mar 16, 2026
@amishamehta99
Copy link
Copy Markdown
Contributor Author

amishamehta99 commented Mar 16, 2026

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
Copy link
Copy Markdown
Member

rcomer commented Mar 16, 2026

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

timhoffm
timhoffm reviewed Mar 16, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@timhoffm timhoffm left a comment

Choose a reason for hiding this comment

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

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

@amishamehta99
Copy link
Copy Markdown
Contributor Author

amishamehta99 commented Mar 16, 2026

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
Copy link
Copy Markdown
Contributor Author

amishamehta99 commented Mar 16, 2026

@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 timhoffm added this to the v3.11.0 milestone Mar 17, 2026
@timhoffm timhoffm merged commit 40730d1 into matplotlib:main Mar 17, 2026
20 checks passed
@timhoffm
Copy link
Copy Markdown
Member

timhoffm commented Mar 17, 2026

Thanks @amishamehta99

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@timhoffm 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

@amishamehta99 @rcomer @timhoffm