don't fail if emptyroom is missing cHPI channels

[drammock]

closes #974

Before merging …

• Changelog has been updated (docs/source/changes.md)

[larsoner]

Looking at:

$ git grep mf_filter
...
mne_bids_pipeline/tests/configs/config_ds004229.py:mf_filter_chpi = False  # for speed, not needed as we low-pass anyway

maybe having this True would have caught the error on main. Do you want to check? If not I can try quickly.

Either way might be good to set to True assuming in only adds a reasonable amount of time on CircleCI (e.g., < 1 minute more is reasonable, > 3 minutes probably isn't).

[larsoner]

maybe having this True would have caught the error on main. Do you want to check? If not I can try quickly.

Locally

rm -Rf ~/mne_data/derivatives/mne-bids-pipeline/ds004229/ && time pytest mne_bids_pipeline/ -k 4229

took 5m50.472s. Setting mf_filter_chpi = True on main did emit an error!

I also realized that we should filter cHPI before performing movement compensation, so I changed that order and added it in the changelog as a bugfix.

Marking for merge-when-green, thanks in advance @drammock !

[drammock]

took 5m50.472s. Setting mf_filter_chpi = True on main did emit an error!

heh, you beat me to it, was just about to check for that failure on main

[larsoner]

Nice 🙂 And just for the sake of completeness, I'll add that changing it to True had a negligible effect on testing time (took 5m49.272s which is a tiny bit shorter than before, so definitely in the test-retest speed standard deviation somewhere).