do we really wanrt to pass epochs data here | throughout the API? I see that this satisfies the sklearn needs, but it seems unnecessary from an mne perspective. I'd be plus one checking whether epochs is ndarrat or instance of mne.Epochs. Please note that epochs[test_idx] will also work + will be less verbose, hence more elegant.