FIX/ENH: more fixes / embellishments

meeg-ml-benchmarks / brain-age-benchmark-paper

M/EEG brain age benchmark paper

https://meeg-ml-benchmarks.github.io/brain-age-benchmark-paper/

BSD 3-Clause "New" or "Revised" License

24 stars 10 forks source link

FIX/ENH: more fixes / embellishments #12

Closed dengemann closed 3 years ago

dengemann commented 3 years ago

Some troubleshooting following #5 ; tuab does not yet work there because we did not deal with custom references; I guess we need to set the average reference right from the beginning to avoid ValueError: Custom EEG reference is not allowed for inverse modeling..

cc @agramfort @gemeinl

dengemann commented 3 years ago

OK as I take a deeper look more issues were hiding under the surface @apmellot

computation broke for most subjects in Cam-CAN, we probably have to handle the rank for MEG
for EEG, no features were computed because of the ValueError: Custom EEG reference is not allowed for inverse modeling.

I should have inspected the feature log files earlier.

dengemann commented 3 years ago

Add this issue here, we'll see if it is necessary https://github.com/dengemann/coffeine/issues/30

dengemann commented 3 years ago

Update: adding the average reference from the beginning fixes the error, it seems we're now getting source localization based features for EEG! More tomorrow.

dengemann commented 3 years ago

Status update: the source pipeline seems to work now. Waiting only for tuab results. All other datasets worked.