Title: [Doc] Clarify edgecolors default for Collection by oscargus · Pull Request #29252 · matplotlib/matplotlib · GitHub
Open Graph Title: [Doc] Clarify edgecolors default for Collection by oscargus · Pull Request #29252 · matplotlib/matplotlib
X Title: [Doc] Clarify edgecolors default for Collection by oscargus · Pull Request #29252 · matplotlib/matplotlib
Description: PR summary This was identified while debugging #29219 There is another case as well: for EventCollection and LineCollection. Seems like it would make more sense to mention it in those classes' documentation. Not sure how it actually effects them though as, e.g., LineCollection says: The properties of each member of a *LineCollection* default to their values in :rc:lines.* instead of :rc:patch.*, and the property colors is added in place of edgecolors. For reference, this is the code: matplotlib/lib/matplotlib/collections.py Lines 813 to 820 in 671177c if c is None: if (mpl.rcParams['patch.force_edgecolor'] or self._edge_default or cbook._str_equal(self._original_facecolor, 'none')): c = self._get_default_edgecolor() else: c = 'none' set_hatch_color = False PR checklist [N/A] "closes #0000" is in the body of the PR description to link the related issue [N/A] new and changed code is tested [N/A] Plotting related features are demonstrated in an example [N/A] New Features and API Changes are noted with a directive and release note Documentation complies with general and docstring guidelines
Open Graph Description: PR summary This was identified while debugging #29219 There is another case as well: for EventCollection and LineCollection. Seems like it would make more sense to mention it in those classes'...
X Description: PR summary This was identified while debugging #29219 There is another case as well: for EventCollection and LineCollection. Seems like it would make more sense to mention it in those classes&...
Opengraph URL: https://github.com/matplotlib/matplotlib/pull/29252
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:c689cb91-edbf-4f94-1ae5-1642cada91ab |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | E484:1A7CA7:1AAB385:23B8727:6A536365 |
| html-safe-nonce | e3e7f93c5c3c83661ab688a66c9d42da4a86f0e79e3ca9a5da2d20030636aa2a |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFNDg0OjFBN0NBNzoxQUFCMzg1OjIzQjg3Mjc6NkE1MzYzNjUiLCJ2aXNpdG9yX2lkIjoiNjI0MDQxODQzNzI1NDA0NjU2NSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 42c345222dd5529dcfcda9f35189ea6e5ab4b9c763c98b488af3793ea26fa374 |
| hovercard-subject-tag | pull_request:2221428607 |
| 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/29252/files |
| twitter:image | https://avatars.githubusercontent.com/u/8114497?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/8114497?s=400&v=4 |
| og:image:alt | PR summary This was identified while debugging #29219 There is another case as well: for EventCollection and LineCollection. Seems like it would make more sense to mention it in those classes'... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | b9a586c06a05a7a86fc7e3f4dbd03e42f6869085879aa184aa6369456dbd50fb |
| 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 |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 07a982c1d40157c619b364352b704c3ce66bb332 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width