Title: [ENH] hatch keyword for pie + some pie documentation by story645 · Pull Request #24470 · matplotlib/matplotlib · GitHub
Open Graph Title: [ENH] hatch keyword for pie + some pie documentation by story645 · Pull Request #24470 · matplotlib/matplotlib
X Title: [ENH] hatch keyword for pie + some pie documentation by story645 · Pull Request #24470 · matplotlib/matplotlib
Description: PR Summary Added the ability to pass in a list of wedgeproperty dicts, one for each wedge of the pie. Follow on to #2923, would probably address #3134 , inspired by twitter convo Yes, I know it can be achieved via loop - see the fig comparison test in this PR - but we were already looping over the wedges inside the function to set all the other properties so the major addition is testing if the input is a dict or list of dicts. <- this could possibly be factored into a cbook function, my guess being 'if thing not list wrap in repeat' is not uncommon in the codebase. Current Iteration Changed this to a hatch kwarg based on the call, added tests + whats new example. cleaned up pct/label loc docs from #24789, reordered docs to mostly match signature, and added sentence that wedgeprop takes precedence over other parameter setting (in current implementation, seemed sorting this out was out of scope for this PR) x = [0.3, 0.3, 0.1] fig, ax = plt.subplots() ax.pie(x, hatch=['/', '+', '.']) which would produce roughly I'll put in a what's new and example if there's consensus on this going in. PR Checklist Tests and Styling Has pytest style unit tests (and pytest passes). Documentation Documentation is sphinx and numpydoc compliant (the docs should build without error). New plotting related features are documented with examples. Release Notes New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/ API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/ Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst
Open Graph Description: PR Summary Added the ability to pass in a list of wedgeproperty dicts, one for each wedge of the pie. Follow on to #2923, would probably address #3134 , inspired by twitter convo Yes, I know it ca...
X Description: PR Summary Added the ability to pass in a list of wedgeproperty dicts, one for each wedge of the pie. Follow on to #2923, would probably address #3134 , inspired by twitter convo Yes, I know it ca...
Opengraph URL: https://github.com/matplotlib/matplotlib/pull/24470
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:38b39573-031e-f63a-d799-c0b32144f809 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | A4B4:204E27:AC79218:AFF6E8B:6A50C02C |
| html-safe-nonce | fc790ae5b91e39890722814a8b8be9dace9897419629b224523d57da1a640fcf |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBNEI0OjIwNEUyNzpBQzc5MjE4OkFGRjZFOEI6NkE1MEMwMkMiLCJ2aXNpdG9yX2lkIjoiNTAzNzM4MTQ0MTc2MTA5OTgyMCIsInJlZ2lvbl9lZGdlIjoic2VhIiwicmVnaW9uX3JlbmRlciI6InNlYSJ9 |
| visitor-hmac | 886e877f38309df4fc379b4ca681d9891f1ad9216817beef807dfcdc77605062 |
| hovercard-subject-tag | pull_request:1123458980 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/matplotlib/matplotlib/pull/24470/files |
| twitter:image | https://avatars.githubusercontent.com/u/1300499?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/1300499?s=400&v=4 |
| og:image:alt | PR Summary Added the ability to pass in a list of wedgeproperty dicts, one for each wedge of the pie. Follow on to #2923, would probably address #3134 , inspired by twitter convo Yes, I know it ca... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 72b13e0e8d0319acdd27a145cfe73f4f06c791713d0ac4690e41fb68ad28cac0 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/matplotlib/matplotlib git https://github.com/matplotlib/matplotlib.git |
| octolytics-dimension-user_id | 215947 |
| octolytics-dimension-user_login | matplotlib |
| octolytics-dimension-repository_id | 1385122 |
| octolytics-dimension-repository_nwo | matplotlib/matplotlib |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 1385122 |
| octolytics-dimension-repository_network_root_nwo | matplotlib/matplotlib |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 08e79d53fc070870d5b168356afebb1e286a2ffe |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width