{"action":"created","issue":{"url":"https://api.github.com/repos/astropy/astropy/issues/6391","repository_url":"https://api.github.com/repos/astropy/astropy","labels_url":"https://api.github.com/repos/astropy/astropy/issues/6391/labels{/name}","comments_url":"https://api.github.com/repos/astropy/astropy/issues/6391/comments","events_url":"https://api.github.com/repos/astropy/astropy/issues/6391/events","html_url":"https://github.com/astropy/astropy/issues/6391","id":245182491,"number":6391,"title":"Wrong Earth radius","user":{"login":"Juanlu001","id":316517,"avatar_url":"https://avatars2.githubusercontent.com/u/316517?v=4","gravatar_id":"","url":"https://api.github.com/users/Juanlu001","html_url":"https://github.com/Juanlu001","followers_url":"https://api.github.com/users/Juanlu001/followers","following_url":"https://api.github.com/users/Juanlu001/following{/other_user}","gists_url":"https://api.github.com/users/Juanlu001/gists{/gist_id}","starred_url":"https://api.github.com/users/Juanlu001/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Juanlu001/subscriptions","organizations_url":"https://api.github.com/users/Juanlu001/orgs","repos_url":"https://api.github.com/users/Juanlu001/repos","events_url":"https://api.github.com/users/Juanlu001/events{/privacy}","received_events_url":"https://api.github.com/users/Juanlu001/received_events","type":"User","site_admin":false},"labels":[{"id":16191481,"url":"https://api.github.com/repos/astropy/astropy/labels/Bug","name":"Bug","color":"eb6420","default":false},{"id":49375858,"url":"https://api.github.com/repos/astropy/astropy/labels/constants","name":"constants","color":"c7def8","default":false}],"state":"open","locked":false,"assignee":{"login":"stargaser","id":970146,"avatar_url":"https://avatars2.githubusercontent.com/u/970146?v=4","gravatar_id":"","url":"https://api.github.com/users/stargaser","html_url":"https://github.com/stargaser","followers_url":"https://api.github.com/users/stargaser/followers","following_url":"https://api.github.com/users/stargaser/following{/other_user}","gists_url":"https://api.github.com/users/stargaser/gists{/gist_id}","starred_url":"https://api.github.com/users/stargaser/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stargaser/subscriptions","organizations_url":"https://api.github.com/users/stargaser/orgs","repos_url":"https://api.github.com/users/stargaser/repos","events_url":"https://api.github.com/users/stargaser/events{/privacy}","received_events_url":"https://api.github.com/users/stargaser/received_events","type":"User","site_admin":false},"assignees":[{"login":"stargaser","id":970146,"avatar_url":"https://avatars2.githubusercontent.com/u/970146?v=4","gravatar_id":"","url":"https://api.github.com/users/stargaser","html_url":"https://github.com/stargaser","followers_url":"https://api.github.com/users/stargaser/followers","following_url":"https://api.github.com/users/stargaser/following{/other_user}","gists_url":"https://api.github.com/users/stargaser/gists{/gist_id}","starred_url":"https://api.github.com/users/stargaser/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stargaser/subscriptions","organizations_url":"https://api.github.com/users/stargaser/orgs","repos_url":"https://api.github.com/users/stargaser/repos","events_url":"https://api.github.com/users/stargaser/events{/privacy}","received_events_url":"https://api.github.com/users/stargaser/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2017-07-24T19:21:55Z","updated_at":"2017-07-25T17:50:24Z","closed_at":null,"body":"As part of #6083 `astropy.constants` was updated, and in particular a new value for the Earth radius, called `\"Nominal Earth equatorial radius\"` and equal to `6.3568e6` was added:\r\n\r\nhttps://github.com/astropy/astropy/blob/v2.0/astropy/constants/iau2015.py#L95\r\n\r\nHowever, according to \"IAU 2015 Resolution B3 on Recommended Nominal Conversion Constants for Selected Solar and Planetary Properties\":\r\n\r\nhttps://arxiv.org/abs/1510.07674\r\n\r\n![screenshot-2017-7-24 - 1510 07674 pdf](https://user-images.githubusercontent.com/316517/28540393-7f1ce650-70b5-11e7-85f5-e8901b7ae6ec.png)\r\n\r\nthe nominal equatorial radius of the Earth is 6.3781e6 m and the nominal polar radius of the Earth is 6.3568e6. Which means that, either the variable should read `\"Nominal Earth polar radius\"` or the value should be changed to `6.3781e6`.\r\n\r\nAlso, from the same report:\r\n\r\n\"If equatorial vs. polar radius is not explicitly specified, it should be understood that nominal terrestrial radius refers specifically to [equatorial], following common usage\".\r\n"},"comment":{"url":"https://api.github.com/repos/astropy/astropy/issues/comments/317816316","html_url":"https://github.com/astropy/astropy/issues/6391#issuecomment-317816316","issue_url":"https://api.github.com/repos/astropy/astropy/issues/6391","id":317816316,"user":{"login":"eteq","id":346587,"avatar_url":"https://avatars1.githubusercontent.com/u/346587?v=4","gravatar_id":"","url":"https://api.github.com/users/eteq","html_url":"https://github.com/eteq","followers_url":"https://api.github.com/users/eteq/followers","following_url":"https://api.github.com/users/eteq/following{/other_user}","gists_url":"https://api.github.com/users/eteq/gists{/gist_id}","starred_url":"https://api.github.com/users/eteq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eteq/subscriptions","organizations_url":"https://api.github.com/users/eteq/orgs","repos_url":"https://api.github.com/users/eteq/repos","events_url":"https://api.github.com/users/eteq/events{/privacy}","received_events_url":"https://api.github.com/users/eteq/received_events","type":"User","site_admin":false},"created_at":"2017-07-25T17:50:24Z","updated_at":"2017-07-25T17:50:24Z","body":"@stargaser - I think this is clearly enough a bug that you can just fix this for 2.0.1 and not have the old one be \"versioned\".  \r\n\r\nIf it were a bit less explicit (e.g. \"Earth radius\"), it's debatable and might want to be versioned, but this is pretty clearly just a typo."}}
{"actor":{"display_login":"eteq"}}