Skip to content
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

DOC: clarify the default value of *radius* in Patch.contains_point #27462

Merged
merged 2 commits into from
Feb 17, 2024

Conversation

tacaswell
Copy link
Member

... and friends.

closes #19807

@tacaswell tacaswell added this to the v3.9.0 milestone Dec 7, 2023
Copy link
Member

@QuLogic QuLogic left a comment

Choose a reason for hiding this comment

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

Ditto on the two other copies.

lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
@tacaswell tacaswell force-pushed the doc/patch_contains_radius_dflt branch from ae5e0c6 to ae79828 Compare December 20, 2023 23:23
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
lib/matplotlib/patches.py Outdated Show resolved Hide resolved
@tacaswell
Copy link
Member Author

Sorry, I was halfway through fixing up the references when I stopped working on this last night.

... and friends.

closes matplotlib#19807

Co-authored-by: Oscar Gustafsson <oscar.gustafsson@gmail.com>
Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
@timhoffm
Copy link
Member

I took the liberty to fix the conflict (via the Github UI, so please squash when merging).

@timhoffm timhoffm merged commit 2fa85ee into matplotlib:main Feb 17, 2024
41 of 42 checks passed
Impaler343 pushed a commit to Impaler343/matplotlib that referenced this pull request Mar 8, 2024
…atplotlib#27462)

... and friends.

closes matplotlib#19807

Co-authored-by: Oscar Gustafsson <oscar.gustafsson@gmail.com>
Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
Co-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>
Impaler343 pushed a commit to Impaler343/matplotlib that referenced this pull request Mar 14, 2024
…atplotlib#27462)

... and friends.

closes matplotlib#19807

Co-authored-by: Oscar Gustafsson <oscar.gustafsson@gmail.com>
Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
Co-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

radius modification in contains_point function when linewidth is specified
5 participants