Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disallowing upper-case EDF extension? #926

Closed
adam2392 opened this issue Nov 11, 2021 · 1 comment
Closed

Disallowing upper-case EDF extension? #926

adam2392 opened this issue Nov 11, 2021 · 1 comment
Labels
EEG Electroencephalography iEEG

Comments

@adam2392
Copy link
Member

Right now, we don't specify what sort of EDF extension to use (upper or lower case). However, it is validated for only lower case extension. In the EDF spec though, both are allowed.

Should we constrain BIDS to only allow lower-case? We need to make it explicit. This makes validation and downstream tool-building easier.

Came up in: https://github.com/bids-standard/bids-validator/pull/1381

@sappelhoff sappelhoff added EEG Electroencephalography iEEG labels Nov 11, 2021
@sappelhoff
Copy link
Member

closed in #927

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EEG Electroencephalography iEEG
Projects
None yet
Development

No branches or pull requests

2 participants