René's URL Explorer Experiment


Title: FIX: correctly handle large arcs by tacaswell · Pull Request #17564 · matplotlib/matplotlib · GitHub

Open Graph Title: FIX: correctly handle large arcs by tacaswell · Pull Request #17564 · matplotlib/matplotlib

X Title: FIX: correctly handle large arcs by tacaswell · Pull Request #17564 · matplotlib/matplotlib

Description: The draw method of mpatches.Arc has two paths: if the arc is "small" compared to its size in the rendered image then render the whole arc and let clipping do it's thing if the arc is "big" compared to its size on the screen then sort out where the circle intersects the axes boundary and only draw that part of it This makes several changes to the Arc draw method: only "squash" the angles in the "small" case is we handle it in the "big" case via transforms make sure that we don't re-order the angles while "squashing" them. Adjusted the test image to use this failing case and to exercise both code paths, but could be convinced to split it out into its own file. closes #17547 PR Checklist Has Pytest style unit tests Code is Flake 8 compliant

Open Graph Description: The draw method of mpatches.Arc has two paths: if the arc is "small" compared to its size in the rendered image then render the whole arc and let clipping do it's thing if the arc is...

X Description: The draw method of mpatches.Arc has two paths: if the arc is "small" compared to its size in the rendered image then render the whole arc and let clipping do it's thing i...

Opengraph URL: https://github.com/matplotlib/matplotlib/pull/17564

X: @github

direct link

Domain: github.com

route-pattern/:user_id/:repository/pull/:id/files(.:format)
route-controllerpull_requests
route-actionfiles
fetch-noncev2:20879717-f330-6ba6-f696-0ff4f88f58dd
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-idE596:95C7C:4A6F6C:642E00:6A540ADF
html-safe-nonceba18e3f2160a8a4594c6cc7d8b6fa38295ba2aca9c277b45a28828fe8e81b972
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFNTk2Ojk1QzdDOjRBNkY2Qzo2NDJFMDA6NkE1NDBBREYiLCJ2aXNpdG9yX2lkIjoiMTg2ODAyMTQxOTMwMjE5Mzg4NyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9
visitor-hmac8ef24ec3885e75561c7577441042f1a248a5b6eb928ac3ea7a68adfd1ef6c98b
hovercard-subject-tagpull_request:427578762
github-keyboard-shortcutsrepository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///pull_requests/show/files
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/matplotlib/matplotlib/pull/17564/files
twitter:imagehttps://avatars.githubusercontent.com/u/199813?s=400&v=4
twitter:cardsummary_large_image
og:imagehttps://avatars.githubusercontent.com/u/199813?s=400&v=4
og:image:altThe draw method of mpatches.Arc has two paths: if the arc is "small" compared to its size in the rendered image then render the whole arc and let clipping do it's thing if the arc is...
og:site_nameGitHub
og:typeobject
hostnamegithub.com
expected-hostnamegithub.com
Noneb9a586c06a05a7a86fc7e3f4dbd03e42f6869085879aa184aa6369456dbd50fb
turbo-cache-controlno-preview
diff-viewunified
go-importgithub.com/matplotlib/matplotlib git https://github.com/matplotlib/matplotlib.git
octolytics-dimension-user_id215947
octolytics-dimension-user_loginmatplotlib
octolytics-dimension-repository_id1385122
octolytics-dimension-repository_nwomatplotlib/matplotlib
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id1385122
octolytics-dimension-repository_network_root_nwomatplotlib/matplotlib
turbo-body-classeslogged-out env-production page-responsive full-width
disable-turbotrue
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
release07a982c1d40157c619b364352b704c3ce66bb332
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/matplotlib/matplotlib/pull/17564/files#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fmatplotlib%2Fmatplotlib%2Fpull%2F17564%2Ffiles
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub Copilot appDirect agents from issue to mergehttps://github.com/features/ai/github-app
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
View all resourceshttps://github.com/resources
GitHub SponsorsFund open source developershttps://github.com/open-source/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/open-source/accelerator
GitHub Starshttps://stars.github.com
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/enterprise/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fmatplotlib%2Fmatplotlib%2Fpull%2F17564%2Ffiles
Sign up https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fpull_requests%2Fshow%2Ffiles&source=header-repo&source_repo=matplotlib%2Fmatplotlib
Reloadhttps://github.com/matplotlib/matplotlib/pull/17564/files
Reloadhttps://github.com/matplotlib/matplotlib/pull/17564/files
Reloadhttps://github.com/matplotlib/matplotlib/pull/17564/files
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
matplotlib https://github.com/matplotlib
matplotlibhttps://github.com/matplotlib/matplotlib
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
Notifications https://github.com/login?return_to=%2Fmatplotlib%2Fmatplotlib
Fork 8.4k https://github.com/login?return_to=%2Fmatplotlib%2Fmatplotlib
Star 23k https://github.com/login?return_to=%2Fmatplotlib%2Fmatplotlib
Code https://github.com/matplotlib/matplotlib
Issues 1.1k https://github.com/matplotlib/matplotlib/issues
Pull requests 408 https://github.com/matplotlib/matplotlib/pulls
Actions https://github.com/matplotlib/matplotlib/actions
Projects https://github.com/matplotlib/matplotlib/projects
Wiki https://github.com/matplotlib/matplotlib/wiki
Security and quality 0 https://github.com/matplotlib/matplotlib/security
Insights https://github.com/matplotlib/matplotlib/pulse
Code https://github.com/matplotlib/matplotlib
Issues https://github.com/matplotlib/matplotlib/issues
Pull requests https://github.com/matplotlib/matplotlib/pulls
Actions https://github.com/matplotlib/matplotlib/actions
Projects https://github.com/matplotlib/matplotlib/projects
Wiki https://github.com/matplotlib/matplotlib/wiki
Security and quality https://github.com/matplotlib/matplotlib/security
Insights https://github.com/matplotlib/matplotlib/pulse
Sign up for GitHub https://github.com/signup?return_to=%2Fmatplotlib%2Fmatplotlib%2Fissues%2Fnew%2Fchoose
terms of servicehttps://docs.github.com/terms
privacy statementhttps://docs.github.com/privacy
Sign inhttps://github.com/login?return_to=%2Fmatplotlib%2Fmatplotlib%2Fissues%2Fnew%2Fchoose
tacaswellhttps://github.com/tacaswell
matplotlib:masterhttps://github.com/matplotlib/matplotlib/tree/master
tacaswell:fix_big_archttps://github.com/tacaswell/matplotlib/tree/fix_big_arc
Conversation 34 https://github.com/matplotlib/matplotlib/pull/17564
Commits 5 https://github.com/matplotlib/matplotlib/pull/17564/commits
Checks 0 https://github.com/matplotlib/matplotlib/pull/17564/checks
Files changed https://github.com/matplotlib/matplotlib/pull/17564/files
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
FIX: correctly handle large arcs https://github.com/matplotlib/matplotlib/pull/17564/files#top
Show all changes 5 commits https://github.com/matplotlib/matplotlib/pull/17564/files
98ab2a0 FIX: correctly handle large arcs tacaswell Jun 3, 2020 https://github.com/matplotlib/matplotlib/pull/17564/commits/98ab2a07920610dae862d95363243572a84ea2c7
a302b5a TST: No longer go through high-resolution code path in test tacaswell Jun 16, 2020 https://github.com/matplotlib/matplotlib/pull/17564/commits/a302b5a1585a0fef44d952ef4318c8ae30b92f26
7ae0c34 MNT: apply no-op exit for non-visible artists tacaswell Jun 11, 2020 https://github.com/matplotlib/matplotlib/pull/17564/commits/7ae0c3438b0e27c4f8693cc6a50c7e73219decb2
d46c161 DOC: remove duplicate word tacaswell Jun 16, 2020 https://github.com/matplotlib/matplotlib/pull/17564/commits/d46c161dac2c0896f897ce7c001f58ee6634dcd7
0dfdfd6 DOC: add more explanatory docstrings tacaswell Jun 16, 2020 https://github.com/matplotlib/matplotlib/pull/17564/commits/0dfdfd68282dab96eb9e36a891fe76457c58bc18
Clear filters https://github.com/matplotlib/matplotlib/pull/17564/files
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
patches.py https://github.com/matplotlib/matplotlib/pull/17564/files#diff-03b74bfefbb7d4eca2804f274825899d191f8bacdbe99ea973d88925f15d1169
arc_angles.png https://github.com/matplotlib/matplotlib/pull/17564/files#diff-e7278e36a92da4e038d1c4d6ca0641e673e7240ff2500b360010123194c190b5
all_quadrants_arcs.svg https://github.com/matplotlib/matplotlib/pull/17564/files#diff-50c5cd87d67e433f1593bca75bf1eb2099de44c3a3fb2a544eba2bba2c5edc6f
large_arc.svg https://github.com/matplotlib/matplotlib/pull/17564/files#diff-f853712f700bb80437bf58e258f5c4da3286bea5e796dbab7f90230580ff51e6
test_patches.py https://github.com/matplotlib/matplotlib/pull/17564/files#diff-87a7e16f4e6ddcd95a4a1041a809c390c03604c0ed5482ea634e1c4927465be5
lib/matplotlib/patches.pyhttps://github.com/matplotlib/matplotlib/pull/17564/files#diff-03b74bfefbb7d4eca2804f274825899d191f8bacdbe99ea973d88925f15d1169
View file https://github.com/tacaswell/matplotlib/blob/0dfdfd68282dab96eb9e36a891fe76457c58bc18/lib/matplotlib/patches.py
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/matplotlib/matplotlib/pull/17564/{{ revealButtonHref }}
https://github.com/matplotlib/matplotlib/pull/17564/files#diff-03b74bfefbb7d4eca2804f274825899d191f8bacdbe99ea973d88925f15d1169
https://github.com/matplotlib/matplotlib/pull/17564/files#diff-03b74bfefbb7d4eca2804f274825899d191f8bacdbe99ea973d88925f15d1169
anntzerhttps://github.com/anntzer
Jun 16, 2020https://github.com/matplotlib/matplotlib/pull/17564/files#r441164004
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
anntzerhttps://github.com/anntzer
Jun 17, 2020https://github.com/matplotlib/matplotlib/pull/17564/files#r441206718
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
tacaswellhttps://github.com/tacaswell
Jun 17, 2020https://github.com/matplotlib/matplotlib/pull/17564/files#r441207898
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
https://github.com/matplotlib/matplotlib/pull/17564/files#diff-03b74bfefbb7d4eca2804f274825899d191f8bacdbe99ea973d88925f15d1169
https://github.com/matplotlib/matplotlib/pull/17564/files#diff-03b74bfefbb7d4eca2804f274825899d191f8bacdbe99ea973d88925f15d1169
https://github.com/matplotlib/matplotlib/pull/17564/files#diff-03b74bfefbb7d4eca2804f274825899d191f8bacdbe99ea973d88925f15d1169
lib/matplotlib/tests/baseline_images/test_axes/arc_angles.pnghttps://github.com/matplotlib/matplotlib/pull/17564/files#diff-e7278e36a92da4e038d1c4d6ca0641e673e7240ff2500b360010123194c190b5
View file https://github.com/tacaswell/matplotlib/blob/0dfdfd68282dab96eb9e36a891fe76457c58bc18/lib/matplotlib/tests/baseline_images/test_axes/arc_angles.png
Open in desktop https://desktop.github.com
Reload?https://github.com/matplotlib/matplotlib/pull/17564/files
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/17564/files
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width


URLs of crawlers that visited me.