Add process to cluster channels based on their proximity to cortical ROI#891
Add process to cluster channels based on their proximity to cortical ROI#891Edouard2laire wants to merge 7 commits intobrainstorm-tools:masterfrom
Conversation
|
Hi @Edouard2laire , Thank you again for the PR and for pushing this forward. Overall, we agree that the approach makes sense and can be useful, particularly for fNIRS applications. In the fNIRS case, clustering channels based on sensitivity and ROI proximity is conceptually consistent with how the modality samples cortical regions, and we think this can be a valuable addition for users working in that framework.
Additionally, since this would be restricted to fNIRS, we are wondering whether it might make more sense for this functionality to be integrated within NIRSTORM rather than the main Brainstorm pipeline. We also plan to run additional tests on different scenarios to better characterize its utility and possible application cases. |
Hi :)
Thanks a lot for the review. I agree that using distance only to cluster the channels would be misleading for EEG and MEG, as the sensitivity profiles of these modalities are fundamentally different and strongly dependent on the head model, montage, and sensor geometry. That's why this PR is also proposing to create the cluster based on the sensitivity map: attributing, for each channel, the ROI for which it has the maximal sensitivity (as estimated by the EEG/MEG forward model). But I agree that it might be difficult to attribute a single region to each sensor if the leadfield spans multiple regions.
I think this is a good idea. We can even put the process under the NIRS tab for now.
I have no objection to this. I prefer to have the process in Brainstorm with a limitation so it works only for NIRS. The reason is that if we move the process to NIRSTORM, then, if a similar process is included in Brainstorm in the future, it will create a duplicate (2 different processes doing the same thing), which might confuse NIRS users. |
|
Thanks @tmedani for the detailed review. I agree with it on the potential confusion that would be introduced for EEG and MEG.
@Edouard2laire, since the process will be only for NIRS, it does make sense to make it part of NIRSTORM. A similar process is not likely to be included in Brainstorm at least for EEG and MEG due to limitations and potential misleading explained by Takfarinas. If you want, I can give it a review once the PR for NIRSTORM is ready. |
|
Closing. Transfered to nirstorm: Nirstorm/nirstorm#302 :) |
This process allows for automatic clustering of channels based on their distance to the ROI on the cortex.
The clustering can be performed either based on Euclidean distance or sensitivity. When using sensitivity, we attribute, for each channel, the ROI with the most sensitivity.
Note: Is there an easy way to visualize only channels from a specific cluster? Or to visualize how the channels are in different clusters on the scalp?
Note 2: I only tested for NIRS, but this should work fine for EEG/MEG.
Note 3: I believe this also satisfies the following request in the to-do list:
Process:
