{"action":"created","issue":{"url":"https://api.github.com/repos/mne-tools/mne-python/issues/3893","repository_url":"https://api.github.com/repos/mne-tools/mne-python","labels_url":"https://api.github.com/repos/mne-tools/mne-python/issues/3893/labels{/name}","comments_url":"https://api.github.com/repos/mne-tools/mne-python/issues/3893/comments","events_url":"https://api.github.com/repos/mne-tools/mne-python/issues/3893/events","html_url":"https://github.com/mne-tools/mne-python/issues/3893","id":199943873,"number":3893,"title":"Add `pick_channels` to montage classes","user":{"login":"choldgraf","id":1839645,"avatar_url":"https://avatars.githubusercontent.com/u/1839645?v=3","gravatar_id":"","url":"https://api.github.com/users/choldgraf","html_url":"https://github.com/choldgraf","followers_url":"https://api.github.com/users/choldgraf/followers","following_url":"https://api.github.com/users/choldgraf/following{/other_user}","gists_url":"https://api.github.com/users/choldgraf/gists{/gist_id}","starred_url":"https://api.github.com/users/choldgraf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/choldgraf/subscriptions","organizations_url":"https://api.github.com/users/choldgraf/orgs","repos_url":"https://api.github.com/users/choldgraf/repos","events_url":"https://api.github.com/users/choldgraf/events{/privacy}","received_events_url":"https://api.github.com/users/choldgraf/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/mne-tools/mne-python/milestones/11","html_url":"https://github.com/mne-tools/mne-python/milestone/11","labels_url":"https://api.github.com/repos/mne-tools/mne-python/milestones/11/labels","id":2233512,"number":11,"title":"0.15","description":"Release by early June in advance of HBM (end of June).","creator":{"login":"Eric89GXL","id":2365790,"avatar_url":"https://avatars.githubusercontent.com/u/2365790?v=3","gravatar_id":"","url":"https://api.github.com/users/Eric89GXL","html_url":"https://github.com/Eric89GXL","followers_url":"https://api.github.com/users/Eric89GXL/followers","following_url":"https://api.github.com/users/Eric89GXL/following{/other_user}","gists_url":"https://api.github.com/users/Eric89GXL/gists{/gist_id}","starred_url":"https://api.github.com/users/Eric89GXL/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Eric89GXL/subscriptions","organizations_url":"https://api.github.com/users/Eric89GXL/orgs","repos_url":"https://api.github.com/users/Eric89GXL/repos","events_url":"https://api.github.com/users/Eric89GXL/events{/privacy}","received_events_url":"https://api.github.com/users/Eric89GXL/received_events","type":"User","site_admin":false},"open_issues":4,"closed_issues":0,"state":"open","created_at":"2017-01-06T17:12:14Z","updated_at":"2017-01-11T16:49:00Z","due_on":"2017-06-02T07:00:00Z","closed_at":null},"comments":7,"created_at":"2017-01-10T21:53:44Z","updated_at":"2017-01-11T17:36:44Z","closed_at":null,"body":"mentioned in #3728 \r\n\r\nbtw I probably shouldn't be the one to implement this PR as I don't have much experience with the philosophy behind montages. Would a `pick_channels` method  just change the `selection` and `ch_names` attributes? And in that case it should probably be a mixin that both `DigMontage` and `Montage` import from.\r\n\r\npinging @agramfort and @Eric89GXL as maybe they have thoughts"},"comment":{"url":"https://api.github.com/repos/mne-tools/mne-python/issues/comments/271936420","html_url":"https://github.com/mne-tools/mne-python/issues/3893#issuecomment-271936420","issue_url":"https://api.github.com/repos/mne-tools/mne-python/issues/3893","id":271936420,"user":{"login":"Eric89GXL","id":2365790,"avatar_url":"https://avatars.githubusercontent.com/u/2365790?v=3","gravatar_id":"","url":"https://api.github.com/users/Eric89GXL","html_url":"https://github.com/Eric89GXL","followers_url":"https://api.github.com/users/Eric89GXL/followers","following_url":"https://api.github.com/users/Eric89GXL/following{/other_user}","gists_url":"https://api.github.com/users/Eric89GXL/gists{/gist_id}","starred_url":"https://api.github.com/users/Eric89GXL/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Eric89GXL/subscriptions","organizations_url":"https://api.github.com/users/Eric89GXL/orgs","repos_url":"https://api.github.com/users/Eric89GXL/repos","events_url":"https://api.github.com/users/Eric89GXL/events{/privacy}","received_events_url":"https://api.github.com/users/Eric89GXL/received_events","type":"User","site_admin":false},"created_at":"2017-01-11T17:36:44Z","updated_at":"2017-01-11T17:36:44Z","body":"Maybe. AFAIK they provide transformation matrices to get between frames -- we do this in some places like `info['dev_head_t']`. But I'm not sure if they have equivalent use cases to what we're going for here, which is to provide locations of specific points in multiple coordinate frames (sometimes the same point in multiple frames, or different points in different frames). Plus we already have something that does this tracking of locations, frames, and point types -- `info['dig'`] is a simple list of dicts that is used all over our code to do this, so there are advantages to staying close to this format."}}
{"actor":{"display_login":"Eric89GXL"}}