FIX: Paths and pyarrow - Githubissues

mne-tools / mne-bids-pipeline

Automatically process entire electrophysiological datasets using MNE-Python.

https://mne.tools/mne-bids-pipeline/

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

134 stars 65 forks source link

FIX: Paths and pyarrow #834

Closed larsoner closed 6 months ago

larsoner commented 6 months ago

Before merging …

[x] Changelog has been updated (docs/source/changes.md)

Some better error messages for path problems, and contains fix for failure in https://github.com/mne-tools/mne-bids-pipeline/pull/833

larsoner commented 6 months ago

@hoechenberger CIs fail with:

RuntimeError: Error 404 when trying to download /home/circleci/mne_data/ds003775/sub-010/ses-t1/sub-010_ses-t1_scans.tsv from https://s3.amazonaws.com/openneuro.org/ds003775/sub-010/ses-t1/sub-010_ses-t1_scans.tsv?versionId=hAC_TNr30jHiLxmLX7ergUVJILPtRjXo

Do we need to open an issue with the openneuro folks about this?

hoechenberger commented 6 months ago

I would guess so :/

larsoner commented 6 months ago

Okay we wait for https://github.com/OpenNeuroOrg/openneuro/issues/2976

larsoner commented 6 months ago

... and I trivially changed us to use ruff-format rather than black for formatting to follow MNE-Python