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
Several bugs when processing #1689
Comments
HenkMutsaerts
added a commit
that referenced
this issue
Apr 21, 2024
HenkMutsaerts
added a commit
that referenced
this issue
Apr 21, 2024
HenkMutsaerts
added a commit
that referenced
this issue
Apr 24, 2024
HenkMutsaerts
added a commit
that referenced
this issue
Apr 24, 2024
HenkMutsaerts
added a commit
that referenced
this issue
Apr 26, 2024
HenkMutsaerts
added a commit
that referenced
this issue
Apr 26, 2024
HenkMutsaerts
added a commit
that referenced
this issue
Apr 26, 2024
HenkMutsaerts
changed the title
Allowing commas in folder names
Several bugs when processing Excersion-VCI
Apr 26, 2024
HenkMutsaerts
changed the title
Several bugs when processing Excersion-VCI
Several bugs when processing
Apr 26, 2024
HenkMutsaerts
added a commit
that referenced
this issue
Apr 29, 2024
HenkMutsaerts
added a commit
that referenced
this issue
Apr 29, 2024
HenkMutsaerts
added a commit
that referenced
this issue
Apr 29, 2024
Open
Open
HenkMutsaerts
added a commit
that referenced
this issue
Apr 30, 2024
HenkMutsaerts
added a commit
that referenced
this issue
Apr 30, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
In some functions we check for a comma ',' as a comma in the extension should be removed: this occurs with spm paths in the format:
dir/filename.nii(.gz),1
dir/filename.nii(.gz),2
dir/filename.nii(.gz),n
for different volumes in a 4D NIfTI file.
When there is a comma in a path (e.g. when mounting), it is useful if we don't have to move data to avoid having to resync it.
Example for me now:
'/Users/hjmutsaerts/surfdrive - Mutsaerts, H.J.M.M. (Henk-Jan)@surfdrive.surf.nl/HolidayPics/studyName'
Matlab otherwise works fine with this path, so spaces, commas, @ don't seem to be a problem in the foldername.
When converting csv to tsv, the comma is converted to a tab, creating another column. The number of columns will be inconsistent and csvtotsv will crash.
We can do two things (or both):
Note that this is only valid for directory names, not for filenames. BIDS subject & visit names should not include illegal chars.
Tasks
How to test
Optional: insert description about how to test the code changes here
Release notes
Required: summarize the changes for the release notes here
The text was updated successfully, but these errors were encountered: