search

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

`openpyxl` dependency not listed #6

Closed gemeinl closed 3 years ago

gemeinl commented 3 years ago

It appears that there is a dependency on openpyxl which is not listed.

agramfort commented 3 years ago

it's in the reqs of the pipeline

https://github.com/mne-tools/mne-bids-pipeline/blob/main/requirements.txt

gemeinl commented 3 years ago

Alright, then this is inconsistent with installation instructions in https://mne.tools/mne-bids-pipeline/getting_started/install.html which was the recommended installation source from the readme page of this repo

agramfort commented 3 years ago

I know it's not up to date ...

@dengemann can you give access to this repo to @hoechenberger so he knows about the pb.

@gemeinl the pipeline is still WIP. Any help to improve it along the way is very welcome...

dengemann commented 3 years ago

Richard should have access already. Lukas has a different assignment in the moment. I will merge this after some testing and integrate this with my local changed. No input needed atm.

From: Alexandre Gramfort @.> Sent: Thursday, October 7, 2021 9:46:23 PM To: dengemann/meeg-brain-age-benchmark-paper @.> Cc: Denis A. Engemann @.>; Mention @.> Subject: Re: [dengemann/meeg-brain-age-benchmark-paper] openpyxl dependency not listed (#6)

I know it's not up to date ...

@dengemannhttps://github.com/dengemann can you give access to this repo to @hoechenbergerhttps://github.com/hoechenberger so he knows about the pb.

@gemeinlhttps://github.com/gemeinl the pipeline is still WIP. Any help to improve it along the way is very welcome...

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/dengemann/meeg-brain-age-benchmark-paper/issues/6#issuecomment-938102997, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAOR7CXRMRFGJROKWO6FD33UFX2I7ANCNFSM5FR6RIDA.

dengemann commented 3 years ago

Richard should have access already. Lukas has a different assignment in the moment. I will merge this after some testing and integrate this with my local changed. No input needed atm. … ____ From: Alexandre Gramfort @.> Sent: Thursday, October 7, 2021 9:46:23 PM To: dengemann/meeg-brain-age-benchmark-paper @.> Cc: Denis A. Engemann @.>; Mention @.> Subject: Re: [dengemann/meeg-brain-age-benchmark-paper] openpyxl dependency not listed (#6) I know it's not up to date ... @dengemannhttps://github.com/dengemann can you give access to this repo to @hoechenbergerhttps://github.com/hoechenberger so he knows about the pb. @gemeinlhttps://github.com/gemeinl the pipeline is still WIP. Any help to improve it along the way is very welcome... — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#6 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAOR7CXRMRFGJROKWO6FD33UFX2I7ANCNFSM5FR6RIDA.

Sorry folks, I somehow misread the email, thinking it belonged to a different PR, please just ignore it.