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

bids_export relies on session number - to fix #231

Open
CPernet opened this issue Nov 13, 2024 · 0 comments
Open

bids_export relies on session number - to fix #231

CPernet opened this issue Nov 13, 2024 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@CPernet
Copy link
Contributor

CPernet commented Nov 13, 2024

where: there are a series of if statements starting @ https://github.com/sccn/EEG-BIDS/blame/master/bids_export.m#L507

expected behaviour: any label is valid for sessions since it is a string in the spec

identified issue: these relies on the length because it assumes number like ses-1 run-2 but one can have ses-baseline and run-1 (runs are ok since defined non negative indices)

possible fix have not figured out

@CPernet CPernet added the bug Something isn't working label Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants