Title: Fix macosx verify later by jklymak · Pull Request #12557 · matplotlib/matplotlib · GitHub
Open Graph Title: Fix macosx verify later by jklymak · Pull Request #12557 · matplotlib/matplotlib
X Title: Fix macosx verify later by jklymak · Pull Request #12557 · matplotlib/matplotlib
Description: PR Summary
This moves the verification of the framework build out of import _macosx, allowing us to
call that more often, and into the creation of the figure manager...
Closes #12188
Update (19 Oct 2018)
Moved MacOSx framework check to pyplot.py so it errors at the same place as before rather than waiting for a draw.
Note this doesn't fix the bouncing icon in the framework build if one indeed wants to use the macosx backend, but it doesn't pop one up everytime regardless of the backend.
Not sure what changed in 3.0 to cause the icon to start bouncing, but this should cure the problem for 75% of the users who just want to use Matplotlib with a different backend, like a notebook...
Icon bouncing is because we initialize the window manager connection to python, but we don't open a window, so the app is in an in-between state of readiness.
Until conda ships a python that we can test if its a FRAMEWORK app, I am not sure we will fix this any better than this. Note the vast improvement here is that the bouncing doesn't happen for every backend like it does in 3.0.0
Non-framework python
this PR
>>> import matplotlib; matplotlib.use('macosx'); import matplotlib.pyplot
yields: No icon popped up and
Traceback (most recent call last):
File "testbackend.py", line 3, in
Open Graph Description: PR Summary This moves the verification of the framework build out of import _macosx, allowing us to call that more often, and into the creation of the figure manager... Closes #12188 Update (19 Oct...
X Description: PR Summary This moves the verification of the framework build out of import _macosx, allowing us to call that more often, and into the creation of the figure manager... Closes #12188 Update (19 Oct...
Opengraph URL: https://github.com/matplotlib/matplotlib/pull/12557
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:6f3b655d-9191-7b58-38a9-b84413618a0b |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | B972:2B5505:8BAD5A:BF615B:6A57068D |
| html-safe-nonce | 767b6fe1a40ba290d567aa82717fd62a8d919a2ad264422c235972168639620f |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCOTcyOjJCNTUwNTo4QkFENUE6QkY2MTVCOjZBNTcwNjhEIiwidmlzaXRvcl9pZCI6IjM1MzAzOTgxODc3ODA1MDcyNzciLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 6c85eb2745ef989801929c1547ecc38daa39609de3fff5b74d0e2b3da3e711da |
| hovercard-subject-tag | pull_request:224061771 |
| 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/12557/files |
| twitter:image | https://avatars.githubusercontent.com/u/1562854?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/1562854?s=400&v=4 |
| og:image:alt | PR Summary This moves the verification of the framework build out of import _macosx, allowing us to call that more often, and into the creation of the figure manager... Closes #12188 Update (19 Oct... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 4eb29e5f807bd95e41196adfbc6d28f9af12d89830d8b684f3e871774ddff2fc |
| 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 | b056788a129fb0194b61f2ebb4c43f6a9f4dfd27 |
| ui-target | canary-2 |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width