{"action":"created","issue":{"url":"https://api.github.com/repos/astropy/astropy/issues/3424","labels_url":"https://api.github.com/repos/astropy/astropy/issues/3424/labels{/name}","comments_url":"https://api.github.com/repos/astropy/astropy/issues/3424/comments","events_url":"https://api.github.com/repos/astropy/astropy/issues/3424/events","html_url":"https://github.com/astropy/astropy/pull/3424","id":56322382,"number":3424,"title":"Observing example improvements","user":{"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},"labels":[{"url":"https://api.github.com/repos/astropy/astropy/labels/Affects-dev","name":"Affects-dev","color":"eb6420"},{"url":"https://api.github.com/repos/astropy/astropy/labels/coordinates","name":"coordinates","color":"c7def8"},{"url":"https://api.github.com/repos/astropy/astropy/labels/Docs","name":"Docs","color":"02d7e1"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/astropy/astropy/milestones/15","labels_url":"https://api.github.com/repos/astropy/astropy/milestones/15/labels","id":643887,"number":15,"title":"v1.0.0","description":"","creator":{"login":"astrofrog","id":314716,"avatar_url":"https://avatars.githubusercontent.com/u/314716?v=3","gravatar_id":"","url":"https://api.github.com/users/astrofrog","html_url":"https://github.com/astrofrog","followers_url":"https://api.github.com/users/astrofrog/followers","following_url":"https://api.github.com/users/astrofrog/following{/other_user}","gists_url":"https://api.github.com/users/astrofrog/gists{/gist_id}","starred_url":"https://api.github.com/users/astrofrog/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/astrofrog/subscriptions","organizations_url":"https://api.github.com/users/astrofrog/orgs","repos_url":"https://api.github.com/users/astrofrog/repos","events_url":"https://api.github.com/users/astrofrog/events{/privacy}","received_events_url":"https://api.github.com/users/astrofrog/received_events","type":"User","site_admin":false},"open_issues":12,"closed_issues":273,"state":"open","created_at":"2014-04-29T16:03:02Z","updated_at":"2015-02-03T21:02:33Z","due_on":"2015-01-23T05:00:00Z","closed_at":null},"comments":3,"created_at":"2015-02-03T01:51:21Z","updated_at":"2015-02-03T21:37:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/astropy/astropy/pulls/3424","html_url":"https://github.com/astropy/astropy/pull/3424","diff_url":"https://github.com/astropy/astropy/pull/3424.diff","patch_url":"https://github.com/astropy/astropy/pull/3424.patch"},"body":"This PR addresses @cdeil's comments from #3217 - it adds more explanation in the observing example to be more searchable, and also makes the warning message not print all of the coordinate data."},"comment":{"url":"https://api.github.com/repos/astropy/astropy/issues/comments/72740360","html_url":"https://github.com/astropy/astropy/pull/3424#issuecomment-72740360","issue_url":"https://api.github.com/repos/astropy/astropy/issues/3424","id":72740360,"user":{"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},"created_at":"2015-02-03T21:37:55Z","updated_at":"2015-02-03T21:37:55Z","body":"@cdeil - my intent with that warning is to emphasize that not all the transformations have been written to support array frame attributes.  I managed to get this one working because it has a clear use case, but some of the other transformation don't necessarily allow this.  Whether we want it to be supported is not clear... for some of the transformations (i.e., those that precess with a fixed transformation matrix), there are substantial performance losses for having vectors of attributes.  \r\n\r\nBut now that you say it, it might be fine to just drop the warning and instead let users encounter exceptions when they try to use it in places where it's not compatible... @astrofrog, what do you think?  Should we remove the warning?"}}