Add the code from PR "Add support for MNE-ICALabel #812"

[jschepers]

Changes

• Added the code from #812 by @hoechenberger
  • Comment: In the newer version of mne-bids-pipeline the ICA is divided into two scripts, therefore I had to divide the code into these scripts.
  • @behinger added two config variables ica_use_ecg_detection and ica_use_eog_detection to control whether MNE eog/ecg detection should be used on the ICs
  • We implemented that the user can specify in the config file (using the variable icalabel_include) which labels/ICs to include. Default is ["brain","other"].
  • Comment: We should add tests for this e.g. No value given -> Default should be used, Empty list or wrong labels -> Error
  • Added plots of the excluded ICs to the report together with their labels and the probability of the labels.
  • Comment: The plotting needs improvement e.g. combine all single plots in one plot

Before merging …

• [ ] Changelog has been updated (docs/source/dev.md.inc)

[welcome[bot]]

Hello! 👋 Thanks for opening your first pull request here! ❤️ We will try to get back to you soon. 🚴🏽‍♂️

[behinger]

thanks for the quick feedback! We will still need some time to work on this, too many parallel things going on.

Because we mainly develop in Julia, not everything is clear to us. I will ask some clarifications on the workflow.

Just foreshadowing two PRs we want to submit: 1) eyelink synchronisation: we understand it is a bit specific, but it is implemented in MNE, so we added a new step to support it (80% ready for PR) 2) zapline: this will be added to the filter thing as a quasi-replacement for 50/60Hz notch filtering, based on meegkit (only just started with the planning of that one).