Bug when running the make_forward script on rest session

[apmellot]

The issue comes from the fact that info is read from evoked file in this script ( info = mne.io.read_info(fname_evoked)) but no evoked files are generated for rest session. @dengemann @agramfort @hoechenberger

[hoechenberger]

We could read it from Raw instead, I suppose?

[dengemann]

Yes. Caution: I suppose you took it from evoked because channels are picked at that point. We may need to think about preserving / translating that logic.

On Fri 13 Aug 2021 at 20:25, Richard Höchenberger @.***> wrote:

We could read it from Raw instead, I suppose?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mne-tools/mne-bids-pipeline/issues/443#issuecomment-898643333, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOR7CRWLJSB6Q2MPJJQAN3T4VPSBANCNFSM5B55YARQ .

[larsoner]

I think this was fixed at some point

https://github.com/mne-tools/mne-bids-pipeline/blob/1cddb952b217ad2d1c8451823842faceeb3afd71/mne_bids_pipeline/scripts/source/_04_make_forward.py#L99

I'm not worried about the channel picking because it's fine for forward to have more channels than evoked, channels should be picked properly automatically when computing the inverse operator for example