{"action":"created","issue":{"url":"https://api.github.com/repos/astropy/astropy/issues/4123","repository_url":"https://api.github.com/repos/astropy/astropy","labels_url":"https://api.github.com/repos/astropy/astropy/issues/4123/labels{/name}","comments_url":"https://api.github.com/repos/astropy/astropy/issues/4123/comments","events_url":"https://api.github.com/repos/astropy/astropy/issues/4123/events","html_url":"https://github.com/astropy/astropy/pull/4123","id":104789640,"number":4123,"title":"Shape changing mixin","user":{"login":"mhvk","id":2789820,"avatar_url":"https://avatars.githubusercontent.com/u/2789820?v=3","gravatar_id":"","url":"https://api.github.com/users/mhvk","html_url":"https://github.com/mhvk","followers_url":"https://api.github.com/users/mhvk/followers","following_url":"https://api.github.com/users/mhvk/following{/other_user}","gists_url":"https://api.github.com/users/mhvk/gists{/gist_id}","starred_url":"https://api.github.com/users/mhvk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mhvk/subscriptions","organizations_url":"https://api.github.com/users/mhvk/orgs","repos_url":"https://api.github.com/users/mhvk/repos","events_url":"https://api.github.com/users/mhvk/events{/privacy}","received_events_url":"https://api.github.com/users/mhvk/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/astropy/astropy/labels/Affects-release","name":"Affects-release","color":"0052cc"},{"url":"https://api.github.com/repos/astropy/astropy/labels/coordinates","name":"coordinates","color":"c7def8"},{"url":"https://api.github.com/repos/astropy/astropy/labels/Ready-for-final-review","name":"Ready-for-final-review","color":"009800"},{"url":"https://api.github.com/repos/astropy/astropy/labels/utils","name":"utils","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/astropy/astropy/milestones/38","html_url":"https://github.com/astropy/astropy/milestones/v1.3.0","labels_url":"https://api.github.com/repos/astropy/astropy/milestones/38/labels","id":1761583,"number":38,"title":"v1.3.0","description":"","creator":{"login":"eteq","id":346587,"avatar_url":"https://avatars.githubusercontent.com/u/346587?v=3","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},"open_issues":20,"closed_issues":22,"state":"open","created_at":"2016-05-12T04:39:02Z","updated_at":"2016-07-20T11:12:54Z","due_on":null,"closed_at":null},"comments":40,"created_at":"2015-09-03T21:57:17Z","updated_at":"2016-07-21T16:04:57Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/astropy/astropy/pulls/4123","html_url":"https://github.com/astropy/astropy/pull/4123","diff_url":"https://github.com/astropy/astropy/pull/4123.diff","patch_url":"https://github.com/astropy/astropy/pull/4123.patch"},"body":"EDIT: this is now a simplied follow-up of #4096, where a new `ShapedLikeNDArray` mixin class is created, which is used in `Time` and in the various coordinate representations. To also use it in frames and coordinates will require that the frame and coordinate attributes are properly broadcast at creating time.\r\n\r\nOLD TEXT FOR REFERENCE\r\nThis is a first cut at #4096, where following @embray's suggestion the shape-changing methods from `Time` are split off to a new `ShapedLikeNDArray` mixin class and used in `Coordinates`. With it,\r\n```\r\nIn [1]: from astropy.coordinates import SkyCoord; import astropy.units as u, numpy as np\r\n\r\nIn [2]: sc = SkyCoord(ra=np.arange(9)*u.deg, dec=np.arange(10, 19)*u.deg)\r\n\r\nIn [3]: sc.reshape(3,3).transpose()\r\nOut[3]: \r\n<SkyCoord (ICRS): (ra, dec) in deg\r\n    [[(0.0, 10.0), (3.0, 13.0), (6.0, 16.0)],\r\n     [(1.0, 11.0), (4.0, 14.0), (7.0, 17.0)],\r\n     [(2.0, 12.0), (5.0, 15.0), (8.0, 18.0)]]>\r\n```\r\n\r\nComments very welcome (as are PRs with test cases; I'd prefer not to have to write them myself; was boring enough for `Time`!). cc: @taldcroft, @eteq.\r\n\r\nNote that there are no test cases yet, though it seems all regular tests pass (which means that `__getitem__` is correctly done). A̶l̶s̶o̶,̶ ̶a̶ ̶p̶r̶o̶b̶l̶e̶m̶ ̶i̶s̶ ̶t̶h̶a̶t̶ ̶`̶s̶c̶.̶T̶`̶ ̶d̶o̶e̶s̶ ̶n̶o̶t̶ ̶w̶o̶r̶k̶ (EDIT: this is solved by the last commit)\r\n"},"comment":{"url":"https://api.github.com/repos/astropy/astropy/issues/comments/234301478","html_url":"https://github.com/astropy/astropy/pull/4123#issuecomment-234301478","issue_url":"https://api.github.com/repos/astropy/astropy/issues/4123","id":234301478,"user":{"login":"StuartLittlefair","id":4570807,"avatar_url":"https://avatars.githubusercontent.com/u/4570807?v=3","gravatar_id":"","url":"https://api.github.com/users/StuartLittlefair","html_url":"https://github.com/StuartLittlefair","followers_url":"https://api.github.com/users/StuartLittlefair/followers","following_url":"https://api.github.com/users/StuartLittlefair/following{/other_user}","gists_url":"https://api.github.com/users/StuartLittlefair/gists{/gist_id}","starred_url":"https://api.github.com/users/StuartLittlefair/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/StuartLittlefair/subscriptions","organizations_url":"https://api.github.com/users/StuartLittlefair/orgs","repos_url":"https://api.github.com/users/StuartLittlefair/repos","events_url":"https://api.github.com/users/StuartLittlefair/events{/privacy}","received_events_url":"https://api.github.com/users/StuartLittlefair/received_events","type":"User","site_admin":false},"created_at":"2016-07-21T16:04:56Z","updated_at":"2016-07-21T16:04:56Z","body":"@mhvk - yes it's currently broken in master too. I'll raise a new issue now..."}}
{"actor":{"display_login":"StuartLittlefair"}}