{"action":"created","issue":{"url":"https://api.github.com/repos/mne-tools/mne-python/issues/1954","labels_url":"https://api.github.com/repos/mne-tools/mne-python/issues/1954/labels{/name}","comments_url":"https://api.github.com/repos/mne-tools/mne-python/issues/1954/comments","events_url":"https://api.github.com/repos/mne-tools/mne-python/issues/1954/events","html_url":"https://github.com/mne-tools/mne-python/issues/1954","id":67894819,"number":1954,"title":"Consolidating list of new functions","user":{"login":"teonlamont","id":1578674,"avatar_url":"https://avatars.githubusercontent.com/u/1578674?v=3","gravatar_id":"","url":"https://api.github.com/users/teonlamont","html_url":"https://github.com/teonlamont","followers_url":"https://api.github.com/users/teonlamont/followers","following_url":"https://api.github.com/users/teonlamont/following{/other_user}","gists_url":"https://api.github.com/users/teonlamont/gists{/gist_id}","starred_url":"https://api.github.com/users/teonlamont/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/teonlamont/subscriptions","organizations_url":"https://api.github.com/users/teonlamont/orgs","repos_url":"https://api.github.com/users/teonlamont/repos","events_url":"https://api.github.com/users/teonlamont/events{/privacy}","received_events_url":"https://api.github.com/users/teonlamont/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/mne-tools/mne-python/milestones/5","html_url":"https://github.com/mne-tools/mne-python/milestones/0.9","labels_url":"https://api.github.com/repos/mne-tools/mne-python/milestones/5/labels","id":1011721,"number":5,"title":"0.9","description":null,"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":15,"closed_issues":89,"state":"open","created_at":"2015-03-09T03:10:39Z","updated_at":"2015-04-22T17:38:06Z","due_on":null,"closed_at":null},"comments":9,"created_at":"2015-04-12T09:43:53Z","updated_at":"2015-04-22T17:39:42Z","closed_at":null,"body":"reiterating some points @dengemann made in #1703. there was a discussion about a poliferation of functions in dev0.9. some may redundant, e.g. `set_channel_positions` and the Montage workflow: `read_montage` and `apply_montage`. I think we should try to hash out what we need and don't. I think before the release, we should definitively clear these before we have a deprecation nightmare in the future. If we want to keep the particular function even though it is very similar to another one, we should make it private. These are the things that come to mind. \r\n\r\nMontages\r\nDigitizer Data\r\nReferencing"},"comment":{"url":"https://api.github.com/repos/mne-tools/mne-python/issues/comments/95279587","html_url":"https://github.com/mne-tools/mne-python/issues/1954#issuecomment-95279587","issue_url":"https://api.github.com/repos/mne-tools/mne-python/issues/1954","id":95279587,"user":{"login":"dengemann","id":1908618,"avatar_url":"https://avatars.githubusercontent.com/u/1908618?v=3","gravatar_id":"","url":"https://api.github.com/users/dengemann","html_url":"https://github.com/dengemann","followers_url":"https://api.github.com/users/dengemann/followers","following_url":"https://api.github.com/users/dengemann/following{/other_user}","gists_url":"https://api.github.com/users/dengemann/gists{/gist_id}","starred_url":"https://api.github.com/users/dengemann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dengemann/subscriptions","organizations_url":"https://api.github.com/users/dengemann/orgs","repos_url":"https://api.github.com/users/dengemann/repos","events_url":"https://api.github.com/users/dengemann/events{/privacy}","received_events_url":"https://api.github.com/users/dengemann/received_events","type":"User","site_admin":false},"created_at":"2015-04-22T17:39:42Z","updated_at":"2015-04-22T17:39:42Z","body":"I would suggest to:\r\n\r\n- making apply_montage private\r\n- add .apply_montage method to our related channels mixin\r\n- make channel pos getters and setters private\r\n"}}