{"action":"created","issue":{"url":"https://api.github.com/repos/mne-tools/mne-python/issues/4947","repository_url":"https://api.github.com/repos/mne-tools/mne-python","labels_url":"https://api.github.com/repos/mne-tools/mne-python/issues/4947/labels{/name}","comments_url":"https://api.github.com/repos/mne-tools/mne-python/issues/4947/comments","events_url":"https://api.github.com/repos/mne-tools/mne-python/issues/4947/events","html_url":"https://github.com/mne-tools/mne-python/issues/4947","id":298087959,"number":4947,"title":"source leakage correction","user":{"login":"dokato","id":4547289,"avatar_url":"https://avatars1.githubusercontent.com/u/4547289?v=4","gravatar_id":"","url":"https://api.github.com/users/dokato","html_url":"https://github.com/dokato","followers_url":"https://api.github.com/users/dokato/followers","following_url":"https://api.github.com/users/dokato/following{/other_user}","gists_url":"https://api.github.com/users/dokato/gists{/gist_id}","starred_url":"https://api.github.com/users/dokato/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dokato/subscriptions","organizations_url":"https://api.github.com/users/dokato/orgs","repos_url":"https://api.github.com/users/dokato/repos","events_url":"https://api.github.com/users/dokato/events{/privacy}","received_events_url":"https://api.github.com/users/dokato/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-02-18T14:17:12Z","updated_at":"2018-02-18T15:55:50Z","closed_at":null,"author_association":"NONE","body":"I don't think that MNE currently supports source leakage correction. According to the work of people from OHBA: https://www.ncbi.nlm.nih.gov/pubmed/27262239, correlation of envelopes after correction is one of the most reliable connectivity estimators.\r\nFor the purpose of my study I implemented symmetric orthogonalization from this paper: https://www.ncbi.nlm.nih.gov/pubmed/25862259\r\nIt's here: https://gist.github.com/dokato/7a997b2a94a0ec6384a5fd0e91e45f8b .\r\nI'm happy to contribute it to mne if you want (but need some advice e.g. on where to place it)."},"comment":{"url":"https://api.github.com/repos/mne-tools/mne-python/issues/comments/366525727","html_url":"https://github.com/mne-tools/mne-python/issues/4947#issuecomment-366525727","issue_url":"https://api.github.com/repos/mne-tools/mne-python/issues/4947","id":366525727,"user":{"login":"dokato","id":4547289,"avatar_url":"https://avatars1.githubusercontent.com/u/4547289?v=4","gravatar_id":"","url":"https://api.github.com/users/dokato","html_url":"https://github.com/dokato","followers_url":"https://api.github.com/users/dokato/followers","following_url":"https://api.github.com/users/dokato/following{/other_user}","gists_url":"https://api.github.com/users/dokato/gists{/gist_id}","starred_url":"https://api.github.com/users/dokato/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dokato/subscriptions","organizations_url":"https://api.github.com/users/dokato/orgs","repos_url":"https://api.github.com/users/dokato/repos","events_url":"https://api.github.com/users/dokato/events{/privacy}","received_events_url":"https://api.github.com/users/dokato/received_events","type":"User","site_admin":false},"created_at":"2018-02-18T15:55:50Z","updated_at":"2018-02-18T15:55:50Z","author_association":"NONE","body":"@dengemann Sure, sounds like a great idea! My implementation requires some amendments obviously, but in my tests worked fine. Let me know if you have any guidelines how to insert it to mne. My guess is that it should be a part of connectivity subpackage, shouldn't it? And then utils, or rather separate module (e.g. `leakage.py`)?"}}
{"actor":{"display_login":"dokato"}}