{"action":"created","issue":{"url":"https://api.github.com/repos/astropy/astropy/issues/4075","labels_url":"https://api.github.com/repos/astropy/astropy/issues/4075/labels{/name}","comments_url":"https://api.github.com/repos/astropy/astropy/issues/4075/comments","events_url":"https://api.github.com/repos/astropy/astropy/issues/4075/events","html_url":"https://github.com/astropy/astropy/pull/4075","id":100878830,"number":4075,"title":"Speed up Column.__getitem__ in general","user":{"login":"embray","id":676149,"avatar_url":"https://avatars.githubusercontent.com/u/676149?v=3","gravatar_id":"","url":"https://api.github.com/users/embray","html_url":"https://github.com/embray","followers_url":"https://api.github.com/users/embray/followers","following_url":"https://api.github.com/users/embray/following{/other_user}","gists_url":"https://api.github.com/users/embray/gists{/gist_id}","starred_url":"https://api.github.com/users/embray/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/embray/subscriptions","organizations_url":"https://api.github.com/users/embray/orgs","repos_url":"https://api.github.com/users/embray/repos","events_url":"https://api.github.com/users/embray/events{/privacy}","received_events_url":"https://api.github.com/users/embray/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/Performance","name":"Performance","color":"bfe5bf"},{"url":"https://api.github.com/repos/astropy/astropy/labels/table","name":"table","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/astropy/astropy/milestones/20","html_url":"https://github.com/astropy/astropy/milestones/v1.1.0","labels_url":"https://api.github.com/repos/astropy/astropy/milestones/20/labels","id":907426,"number":20,"title":"v1.1.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":8,"closed_issues":161,"state":"open","created_at":"2014-12-17T20:05:23Z","updated_at":"2015-10-05T16:49:49Z","due_on":"2015-07-23T04:00:00Z","closed_at":null},"comments":17,"created_at":"2015-08-13T21:55:05Z","updated_at":"2015-10-05T17:21:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/astropy/astropy/pulls/4075","html_url":"https://github.com/astropy/astropy/pull/4075","diff_url":"https://github.com/astropy/astropy/pull/4075.diff","patch_url":"https://github.com/astropy/astropy/pull/4075.patch"},"body":"Alternative to #3930--speed up `Column.__getitem__` using Cython base classes.  This eliminates most of the overhead associated with calling pure-Python `__getitem__`, and speeds up the body of the method as well.\r\n\r\nThis is a more general solution than the one in #3930--I think this could be useful for #3915.  Checking whether the indices need to be updated adds only a trivial amount of overhead, so we can perform that check in the actual `__getitem__` without incurring additional overhead.  So unless we still want to be able to use `table[slice]` without incurring index adjustment overhead, the `get_item` methods won't be necessary."},"comment":{"url":"https://api.github.com/repos/astropy/astropy/issues/comments/145603860","html_url":"https://github.com/astropy/astropy/pull/4075#issuecomment-145603860","issue_url":"https://api.github.com/repos/astropy/astropy/issues/4075","id":145603860,"user":{"login":"taldcroft","id":348089,"avatar_url":"https://avatars.githubusercontent.com/u/348089?v=3","gravatar_id":"","url":"https://api.github.com/users/taldcroft","html_url":"https://github.com/taldcroft","followers_url":"https://api.github.com/users/taldcroft/followers","following_url":"https://api.github.com/users/taldcroft/following{/other_user}","gists_url":"https://api.github.com/users/taldcroft/gists{/gist_id}","starred_url":"https://api.github.com/users/taldcroft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/taldcroft/subscriptions","organizations_url":"https://api.github.com/users/taldcroft/orgs","repos_url":"https://api.github.com/users/taldcroft/repos","events_url":"https://api.github.com/users/taldcroft/events{/privacy}","received_events_url":"https://api.github.com/users/taldcroft/received_events","type":"User","site_admin":false},"created_at":"2015-10-05T17:21:09Z","updated_at":"2015-10-05T17:21:09Z","body":"@embray - that's definitely in the noise, so no problem!  Thanks for working on this."}}