Title: Drop support for directly imread()ing urls. · Issue #18648 · matplotlib/matplotlib · GitHub
Open Graph Title: Drop support for directly imread()ing urls. · Issue #18648 · matplotlib/matplotlib
X Title: Drop support for directly imread()ing urls. · Issue #18648 · matplotlib/matplotlib
Description: Describe the issue Summary Support for directly imread()ing url strings caused Matplotlib to add a dependency on a package which we should not have no business of having a dependency on (certifi, see #18636 (comment)). url strings can't ...
Open Graph Description: Describe the issue Summary Support for directly imread()ing url strings caused Matplotlib to add a dependency on a package which we should not have no business of having a dependency on (certifi, s...
X Description: Describe the issue Summary Support for directly imread()ing url strings caused Matplotlib to add a dependency on a package which we should not have no business of having a dependency on (certifi, s...
Opengraph URL: https://github.com/matplotlib/matplotlib/issues/18648
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Drop support for directly imread()ing urls.","articleBody":"### Describe the issue\r\n\r\n**Summary**\r\n\r\nSupport for directly imread()ing url strings caused Matplotlib to add a dependency on a package which we should not have no business of having a dependency on (`certifi`, see https://github.com/matplotlib/matplotlib/pull/18636#issuecomment-703126749). url strings can't be directly `open`'ed either, so if we instead documented the (easy) solution of calling `urllib.request.urlopen` on the URL (+/- certifi) it'd be much more educative for users (now they know how to urlopen anything, not just images) (note that one of the main reasons for originally adding support to directly opening url strings was that the name of `urllib.request.urlopen` changed between Py2 and Py3, https://github.com/matplotlib/matplotlib/pull/4256#issuecomment-84369830, but that's obviously not relevant anymore). Finally if we really want that functionality it would make much more sense to have it in Pillow anyways (given that imread is just a thin wrapper around PIL.Image.open at that point).\r\n\r\n### Proposed fix\r\n\r\nDeprecate and then drop support for `imread(\"https://...\")`.\r\n\r\n","author":{"url":"https://github.com/anntzer","@type":"Person","name":"anntzer"},"datePublished":"2020-10-03T16:17:14.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/18648/matplotlib/issues/18648"}
| 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:72ccd1c2-3acc-8a80-322e-3ebc423df891 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | CA34:3F7705:35974A:4AF858:6A537361 |
| html-safe-nonce | c7c7e408d1caf1de98f3bc424a092ebacb6242ed3148355080cb0922f96bfb62 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDQTM0OjNGNzcwNTozNTk3NEE6NEFGODU4OjZBNTM3MzYxIiwidmlzaXRvcl9pZCI6IjE0ODEzOTIzNDE2NzE3MDMzOTMiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 506fe2a41afbc1f7e02de34a7f474030a307437e3a78dc50337eb92da94217ec |
| hovercard-subject-tag | issue:714123098 |
| 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/18648/issue_layout |
| twitter:image | https://opengraph.githubassets.com/f1577f83619db8142c7456d0ba3141838db18258dad65933630d2b64915378a6/matplotlib/matplotlib/issues/18648 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/f1577f83619db8142c7456d0ba3141838db18258dad65933630d2b64915378a6/matplotlib/matplotlib/issues/18648 |
| og:image:alt | Describe the issue Summary Support for directly imread()ing url strings caused Matplotlib to add a dependency on a package which we should not have no business of having a dependency on (certifi, s... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | anntzer |
| hostname | github.com |
| expected-hostname | github.com |
| None | b9a586c06a05a7a86fc7e3f4dbd03e42f6869085879aa184aa6369456dbd50fb |
| 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 | 07a982c1d40157c619b364352b704c3ce66bb332 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width