mne-tools / mne-connectivity

Connectivity algorithms that leverage the MNE-Python API.
https://mne.tools/mne-connectivity/dev/index.html
BSD 3-Clause "New" or "Revised" License
68 stars 34 forks source link

ENH: Add symmetric orthogonalization #36

Closed larsoner closed 3 years ago

larsoner commented 3 years ago

This implements symmetric orthogonalization following https://www.sciencedirect.com/science/article/pii/S1053811915002670.

@agramfort @dengemann @SherazKhan any thoughts on the utility of this versus the already-implemented pairwise orthogonalization? It seems to have the limitation of requiring n_labels < rank, but some advantages mentioned in the paper above (better sensitivity / fewer false alarms among them).

dengemann commented 3 years ago

@larsoner cool! Once it is implemented we should try to understand its benefits :)

larsoner commented 3 years ago

Once it is implemented we should try to understand its benefits :)

It's implemented here, feel free to try it out on some data

SherazKhan commented 3 years ago

This method is indeed valid and has been used in multiple application papers.

larsoner commented 3 years ago

https://253-369628653-gh.circle-artifacts.com/0/dev/auto_examples/mne_inverse_envelope_correlation.html