{"action":"created","issue":{"url":"https://api.github.com/repos/matplotlib/matplotlib/issues/10789","repository_url":"https://api.github.com/repos/matplotlib/matplotlib","labels_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/10789/labels{/name}","comments_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/10789/comments","events_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/10789/events","html_url":"https://github.com/matplotlib/matplotlib/issues/10789","id":305322061,"number":10789,"title":"Make pie charts circular by default","user":{"login":"nicoguaro","id":1097787,"avatar_url":"https://avatars2.githubusercontent.com/u/1097787?v=4","gravatar_id":"","url":"https://api.github.com/users/nicoguaro","html_url":"https://github.com/nicoguaro","followers_url":"https://api.github.com/users/nicoguaro/followers","following_url":"https://api.github.com/users/nicoguaro/following{/other_user}","gists_url":"https://api.github.com/users/nicoguaro/gists{/gist_id}","starred_url":"https://api.github.com/users/nicoguaro/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nicoguaro/subscriptions","organizations_url":"https://api.github.com/users/nicoguaro/orgs","repos_url":"https://api.github.com/users/nicoguaro/repos","events_url":"https://api.github.com/users/nicoguaro/events{/privacy}","received_events_url":"https://api.github.com/users/nicoguaro/received_events","type":"User","site_admin":false},"labels":[{"id":75481695,"url":"https://api.github.com/repos/matplotlib/matplotlib/labels/default%20changes","name":"default changes","color":"fbca04","default":false},{"id":720838099,"url":"https://api.github.com/repos/matplotlib/matplotlib/labels/good%20first%20issue","name":"good first issue","color":"5319e7","default":true},{"id":115014,"url":"https://api.github.com/repos/matplotlib/matplotlib/labels/wishlist","name":"wishlist","color":"02d7e1","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-03-14T20:45:48Z","updated_at":"2018-03-26T09:53:18Z","closed_at":null,"author_association":"NONE","body":"<!--To help us understand and resolve your issue, please fill out the form to the best of your ability.-->\r\n<!--You can feel free to delete the sections that do not apply.-->\r\n\r\n### Bug report\r\n\r\n**Bug summary**\r\n\r\nThe default configuration for pie charts should give circular pies. That is in the definition according to Wikipedia:\r\n\r\n> A pie chart (or a circle chart) is a circular statistical graphic which is divided into slices to illustrate numerical proportion. \r\n\r\nSee for example: https://kosara.net/papers/2016/Kosara-EuroVis-2016.pdf\r\n\r\n\r\n**Code for reproduction**\r\n\r\n<!--A minimum code snippet required to reproduce the bug, also minimizing the number of dependencies required-->\r\n\r\n```python\r\nimport matplotlib.pyplot as plt\r\n\r\nplt.pie([10, 90])\r\nplt.show()\r\n```\r\n\r\n![pie_ellipse](https://user-images.githubusercontent.com/1097787/37429625-e6631984-279d-11e8-8ff1-e029d6a3229c.png)\r\n\r\n**Actual outcome**\r\n\r\n\r\n\r\n**Expected outcome**\r\n\r\nThe expected outcome is the following\r\n\r\n![pie_circle](https://user-images.githubusercontent.com/1097787/37429677-11f21438-279e-11e8-9b7e-0e930fd176dc.png)\r\n\r\nThis is obtained by the following snippet\r\n\r\n````python\r\nimport matplotlib.pyplot as plt\r\n\r\nplt.pie([10, 90])\r\nplt.axis(\"image\")\r\nplt.show()\r\n````\r\n\r\n**Matplotlib version**\r\n<!--Please specify your platform and versions of the relevant libraries you are using:-->\r\n  * Operating system: Linux Mint\r\n  * Matplotlib version: 2.1.2\r\n  * Matplotlib backend (`print(matplotlib.get_backend())`): Qt5Agg\r\n  * Python version: 3.6.4\r\n  * Distribution: Anaconda custom (64-bit)| (default, Jan 16 2018, 18:10:19)\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/matplotlib/matplotlib/issues/comments/376111104","html_url":"https://github.com/matplotlib/matplotlib/issues/10789#issuecomment-376111104","issue_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/10789","id":376111104,"user":{"login":"Imanflow","id":17592238,"avatar_url":"https://avatars0.githubusercontent.com/u/17592238?v=4","gravatar_id":"","url":"https://api.github.com/users/Imanflow","html_url":"https://github.com/Imanflow","followers_url":"https://api.github.com/users/Imanflow/followers","following_url":"https://api.github.com/users/Imanflow/following{/other_user}","gists_url":"https://api.github.com/users/Imanflow/gists{/gist_id}","starred_url":"https://api.github.com/users/Imanflow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Imanflow/subscriptions","organizations_url":"https://api.github.com/users/Imanflow/orgs","repos_url":"https://api.github.com/users/Imanflow/repos","events_url":"https://api.github.com/users/Imanflow/events{/privacy}","received_events_url":"https://api.github.com/users/Imanflow/received_events","type":"User","site_admin":false},"created_at":"2018-03-26T09:53:18Z","updated_at":"2018-03-26T09:53:18Z","author_association":"NONE","body":"I saw this tagged as a good first issue and I started looking at it. Not only using `plt.axis(\"image\")` works, also with `equal` and ` scaled`. \r\n\r\nI checked the code in `pyplot.py` and `_axes.py` and I fail to see clearly what should be modified in order to make it by default, but with a bit of guidance I would take the issue."}}
{"actor":{"display_login":"Imanflow"}}