{"action":"created","issue":{"url":"https://api.github.com/repos/matplotlib/matplotlib/issues/6281","repository_url":"https://api.github.com/repos/matplotlib/matplotlib","labels_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/6281/labels{/name}","comments_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/6281/comments","events_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/6281/events","html_url":"https://github.com/matplotlib/matplotlib/pull/6281","id":146815944,"number":6281,"title":"FIX: do not always reset scales from figureoptions","user":{"login":"tacaswell","id":199813,"avatar_url":"https://avatars.githubusercontent.com/u/199813?v=3","gravatar_id":"","url":"https://api.github.com/users/tacaswell","html_url":"https://github.com/tacaswell","followers_url":"https://api.github.com/users/tacaswell/followers","following_url":"https://api.github.com/users/tacaswell/following{/other_user}","gists_url":"https://api.github.com/users/tacaswell/gists{/gist_id}","starred_url":"https://api.github.com/users/tacaswell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tacaswell/subscriptions","organizations_url":"https://api.github.com/users/tacaswell/orgs","repos_url":"https://api.github.com/users/tacaswell/repos","events_url":"https://api.github.com/users/tacaswell/events{/privacy}","received_events_url":"https://api.github.com/users/tacaswell/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/matplotlib/matplotlib/labels/needs_review","name":"needs_review","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/matplotlib/matplotlib/milestones/22","html_url":"https://github.com/matplotlib/matplotlib/milestones/1.5.2%20(Critical%20bug%20fix%20release)","labels_url":"https://api.github.com/repos/matplotlib/matplotlib/milestones/22/labels","id":1495785,"number":22,"title":"1.5.2 (Critical bug fix release)","description":"Speculative milestone is case we need to do a 1.5.2 release. ","creator":{"login":"tacaswell","id":199813,"avatar_url":"https://avatars.githubusercontent.com/u/199813?v=3","gravatar_id":"","url":"https://api.github.com/users/tacaswell","html_url":"https://github.com/tacaswell","followers_url":"https://api.github.com/users/tacaswell/followers","following_url":"https://api.github.com/users/tacaswell/following{/other_user}","gists_url":"https://api.github.com/users/tacaswell/gists{/gist_id}","starred_url":"https://api.github.com/users/tacaswell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tacaswell/subscriptions","organizations_url":"https://api.github.com/users/tacaswell/orgs","repos_url":"https://api.github.com/users/tacaswell/repos","events_url":"https://api.github.com/users/tacaswell/events{/privacy}","received_events_url":"https://api.github.com/users/tacaswell/received_events","type":"User","site_admin":false},"open_issues":36,"closed_issues":103,"state":"open","created_at":"2016-01-10T22:41:29Z","updated_at":"2016-04-29T01:54:23Z","due_on":null,"closed_at":null},"comments":3,"created_at":"2016-04-08T05:14:06Z","updated_at":"2016-04-29T12:20:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/matplotlib/matplotlib/pulls/6281","html_url":"https://github.com/matplotlib/matplotlib/pull/6281","diff_url":"https://github.com/matplotlib/matplotlib/pull/6281.diff","patch_url":"https://github.com/matplotlib/matplotlib/pull/6281.patch"},"body":"Only try to set the axis scales if they have been changed.\r\n\r\ncloses #6276\r\n\r\nThe `set_yscale` and `set_xscale` Axes methods call out to\r\n`Axis_set_scale` which resets the default locator/formatters (which\r\nmakes sense when flipping between log/linear).\r\n\r\nPutting the cut-out in `figureoption` is the less disruptive change,\r\neven if the case could be made that the no-op short-circuit should be\r\ndone in Axis or Axes."},"comment":{"url":"https://api.github.com/repos/matplotlib/matplotlib/issues/comments/215696853","html_url":"https://github.com/matplotlib/matplotlib/pull/6281#issuecomment-215696853","issue_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/6281","id":215696853,"user":{"login":"tacaswell","id":199813,"avatar_url":"https://avatars.githubusercontent.com/u/199813?v=3","gravatar_id":"","url":"https://api.github.com/users/tacaswell","html_url":"https://github.com/tacaswell","followers_url":"https://api.github.com/users/tacaswell/followers","following_url":"https://api.github.com/users/tacaswell/following{/other_user}","gists_url":"https://api.github.com/users/tacaswell/gists{/gist_id}","starred_url":"https://api.github.com/users/tacaswell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tacaswell/subscriptions","organizations_url":"https://api.github.com/users/tacaswell/orgs","repos_url":"https://api.github.com/users/tacaswell/repos","events_url":"https://api.github.com/users/tacaswell/events{/privacy}","received_events_url":"https://api.github.com/users/tacaswell/received_events","type":"User","site_admin":false},"created_at":"2016-04-29T12:20:48Z","updated_at":"2016-04-29T12:20:48Z","body":"That definitely feels like overengineering to me.  If someone did want to keep track of the full history of the formatters/locators then it should live in a layer above base mpl."}}