payload: {"action":"created","issue":{"url":"https://api.github.com/repos/matplotlib/matplotlib/issues/13110","repository_url":"https://api.github.com/repos/matplotlib/matplotlib","labels_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/13110/labels{/name}","comments_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/13110/comments","events_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/13110/events","html_url":"https://github.com/matplotlib/matplotlib/pull/13110","id":396188286,"node_id":"MDExOlB1bGxSZXF1ZXN0MjQyNDUzNDU5","number":13110,"title":"Slightly tighten the Bbox/Transform API.","user":{"login":"anntzer","id":1322974,"node_id":"MDQ6VXNlcjEzMjI5NzQ=","avatar_url":"https://avatars0.githubusercontent.com/u/1322974?v=4","gravatar_id":"","url":"https://api.github.com/users/anntzer","html_url":"https://github.com/anntzer","followers_url":"https://api.github.com/users/anntzer/followers","following_url":"https://api.github.com/users/anntzer/following{/other_user}","gists_url":"https://api.github.com/users/anntzer/gists{/gist_id}","starred_url":"https://api.github.com/users/anntzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anntzer/subscriptions","organizations_url":"https://api.github.com/users/anntzer/orgs","repos_url":"https://api.github.com/users/anntzer/repos","events_url":"https://api.github.com/users/anntzer/events{/privacy}","received_events_url":"https://api.github.com/users/anntzer/received_events","type":"User","site_admin":false},"labels":[{"id":815558876,"node_id":"MDU6TGFiZWw4MTU1NTg4NzY=","url":"https://api.github.com/repos/matplotlib/matplotlib/labels/Needs%20rebase","name":"Needs rebase","color":"e27e6a","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2019-01-05T18:22:50Z","updated_at":"2019-10-01T17:09:58Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/matplotlib/matplotlib/pulls/13110","html_url":"https://github.com/matplotlib/matplotlib/pull/13110","diff_url":"https://github.com/matplotlib/matplotlib/pull/13110.diff","patch_url":"https://github.com/matplotlib/matplotlib/pull/13110.patch"},"body":"At least it's explicit that Bbox.inverse_transformed() follows the\r\nBbox.transformed() semantics, not those of Transform.transform_bbox() or\r\nof TransformedBbox (#12059) (sounds obvious, but you never know...).\r\n\r\nAlso one less thing to implement for a possible future C implementation of the transform stack.\r\n\r\n## PR Summary\r\n\r\n## PR Checklist\r\n\r\n- [ ] Has Pytest style unit tests\r\n- [ ] Code is [Flake 8](http://flake8.pycqa.org/en/latest/) compliant\r\n- [ ] New features are documented, with examples if plot related\r\n- [ ] Documentation is sphinx and numpydoc compliant\r\n- [ ] Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)\r\n- [ ] Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way\r\n\r\n<!--\r\nThank you so much for your PR!  To help us review your contribution, please\r\nconsider the following points:\r\n\r\n- A development guide is available at https://matplotlib.org/devdocs/devel/index.html.\r\n\r\n- Help with git and github is available at\r\n  https://matplotlib.org/devel/gitwash/development_workflow.html.\r\n\r\n- Do not create the PR out of master, but out of a separate branch.\r\n\r\n- The PR title should summarize the changes, for example \"Raise ValueError on\r\n  non-numeric input to set_xlim\".  Avoid non-descriptive titles such as\r\n  \"Addresses issue #8576\".\r\n\r\n- The summary should provide at least 1-2 sentences describing the pull request\r\n  in detail (Why is this change required?  What problem does it solve?) and\r\n  link to any relevant issues.\r\n\r\n- If you are contributing fixes to docstrings, please pay attention to\r\n  http://matplotlib.org/devel/documenting_mpl.html#formatting.  In particular,\r\n  note the difference between using single backquotes, double backquotes, and\r\n  asterisks in the markup.\r\n\r\nWe understand that PRs can sometimes be overwhelming, especially as the\r\nreviews start coming in.  Please let us know if the reviews are unclear or\r\nthe recommended next step seems overly demanding, if you would like help in\r\naddressing a reviewer's comments, or if you have been waiting too long to hear\r\nback on your PR.\r\n-->\r\n"},"comment":{"url":"https://api.github.com/repos/matplotlib/matplotlib/issues/comments/537134207","html_url":"https://github.com/matplotlib/matplotlib/pull/13110#issuecomment-537134207","issue_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/13110","id":537134207,"node_id":"MDEyOklzc3VlQ29tbWVudDUzNzEzNDIwNw==","user":{"login":"anntzer","id":1322974,"node_id":"MDQ6VXNlcjEzMjI5NzQ=","avatar_url":"https://avatars0.githubusercontent.com/u/1322974?v=4","gravatar_id":"","url":"https://api.github.com/users/anntzer","html_url":"https://github.com/anntzer","followers_url":"https://api.github.com/users/anntzer/followers","following_url":"https://api.github.com/users/anntzer/following{/other_user}","gists_url":"https://api.github.com/users/anntzer/gists{/gist_id}","starred_url":"https://api.github.com/users/anntzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anntzer/subscriptions","organizations_url":"https://api.github.com/users/anntzer/orgs","repos_url":"https://api.github.com/users/anntzer/repos","events_url":"https://api.github.com/users/anntzer/events{/privacy}","received_events_url":"https://api.github.com/users/anntzer/received_events","type":"User","site_admin":false},"created_at":"2019-10-01T17:09:58Z","updated_at":"2019-10-01T17:09:58Z","author_association":"CONTRIBUTOR","body":"Do you know the difference between TransformedBbox, Transform.transform_bbox, and Bbox.transformed?  They all do slightly different things (#12059), and I think it's too bad that this is not more obvious from the names of the APIs; in other words there are too many ways to \"apply\" a transform to a bbox, and I'm trying to start removing some of those."}}
other: {"actor":{"display_login":"anntzer"}}