Title: MatplotlibDeprecationWarning removal hard-breaks seaborn in 3.6rc1 · Issue #23716 · matplotlib/matplotlib · GitHub
Open Graph Title: MatplotlibDeprecationWarning removal hard-breaks seaborn in 3.6rc1 · Issue #23716 · matplotlib/matplotlib
X Title: MatplotlibDeprecationWarning removal hard-breaks seaborn in 3.6rc1 · Issue #23716 · matplotlib/matplotlib
Description: This removal hard breaks a seaborn function. Also it's rather unlikely for end users to import MatplotlibDeprecationWarning. Curious what the basis for this claim was? https://github.com/search?q="from+matplotlib.cbook+import+Matplotli...
Open Graph Description: This removal hard breaks a seaborn function. Also it's rather unlikely for end users to import MatplotlibDeprecationWarning. Curious what the basis for this claim was? https://github.com/search?q=%...
X Description: This removal hard breaks a seaborn function. Also it's rather unlikely for end users to import MatplotlibDeprecationWarning. Curious what the basis for this claim was? https://github.com/search...
Opengraph URL: https://github.com/matplotlib/matplotlib/issues/23716
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"MatplotlibDeprecationWarning removal hard-breaks seaborn in 3.6rc1","articleBody":"This removal hard breaks a seaborn function.\r\n\r\n\u003e Also it's rather unlikely for end users to import MatplotlibDeprecationWarning.\r\n\r\nCurious what the basis for this claim was? https://github.com/search?q=%22from+matplotlib.cbook+import+MatplotlibDeprecationWarning%22\u0026type=Code shows 2k+ direct imports of this class, and searching for alternate usage patterns turns up others as well.\r\n\r\n\u003e I don't think we have or easily could have added a warning on importing the class\r\n\r\nI agree that this would have been hard! But IMO that puts the burden on matplotlib to deprecate with a bit more care (e.g., proactively searching for usage in downstream libraries, etc.). You are right that this was \"properly documented\", but it is mentioned in the middle of a very long list of API changes (there are \u003e4000 words on https://matplotlib.org/stable/api/prev_api_changes/api_changes_3.4.0.html). Downstream libraries have the reasonable expectation that they can rely on unit test coverage to surface deprecation warnings, since matplotlib is usually good about that sort of thing.\r\n\r\nIn the future, please boost the signal of any silent deprecations.\r\n\r\n_Originally posted by @mwaskom in https://github.com/matplotlib/matplotlib/issues/22514#issuecomment-1223907658_","author":{"url":"https://github.com/tacaswell","@type":"Person","name":"tacaswell"},"datePublished":"2022-08-23T14:25:16.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":5},"url":"https://github.com/23716/matplotlib/issues/23716"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:21b540f2-4ab1-5273-b4f3-0e566f4b1328 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | B45A:19D5B9:377F53:4B1414:6A5140CD |
| html-safe-nonce | 7ad220b20707c978285a2ea109f3ff54689e6a91ab2d552d18c506d806efc327 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCNDVBOjE5RDVCOTozNzdGNTM6NEIxNDE0OjZBNTE0MENEIiwidmlzaXRvcl9pZCI6IjUwNjI0MzEyMDI4NTk4OTI5NDEiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 6d4277127724a910e173aa8f734e7def471bbaec3289c91f4aaf612a3c9d0c7b |
| hovercard-subject-tag | issue:1348047093 |
| github-keyboard-shortcuts | repository,issues,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/_view_fragments/issues/show/matplotlib/matplotlib/23716/issue_layout |
| twitter:image | https://opengraph.githubassets.com/b0ade3d52ae6dff54085aa8528fd74618f5fbbf72dc4cefb75eb9cb58a8560d8/matplotlib/matplotlib/issues/23716 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/b0ade3d52ae6dff54085aa8528fd74618f5fbbf72dc4cefb75eb9cb58a8560d8/matplotlib/matplotlib/issues/23716 |
| og:image:alt | This removal hard breaks a seaborn function. Also it's rather unlikely for end users to import MatplotlibDeprecationWarning. Curious what the basis for this claim was? https://github.com/search?q=%... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | tacaswell |
| hostname | github.com |
| expected-hostname | github.com |
| None | e840f405b2718e8f2d55aafa9ff27dbce17e29d0c253011d05ea0fea3c78baff |
| turbo-cache-control | no-preview |
| 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 | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | e876f00c5723f6080b9d294e4958fd4d506e6faf |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width