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

Checking participants against folder names #1901

Open
arnodelorme opened this issue Feb 27, 2024 · 3 comments
Open

Checking participants against folder names #1901

arnodelorme opened this issue Feb 27, 2024 · 3 comments
Labels

Comments

@arnodelorme
Copy link

The participants.tsv file does not contain the same subject ID as the folder name. One may reasonably assume that the subjects are in the same order as the folder in the participants.tsv file. However, since this cannot be confirmed, this file should be processed with caution. It should probably not have passed the BIDS validator.

https://nemar.org/dataexplorer/detail?dataset_id=ds002712

@Remi-Gau
Copy link
Contributor

the participant.tsv is not formatted according to BIDS (participant_id should be sub-something), so I don't think this is a validator issue.

image

@arnodelorme
Copy link
Author

arnodelorme commented Feb 28, 2024

Well, isn't that an issue that we cannot reconcile participant ID with subject ID? @effigies this is the dataset we discussed 2 months ago on OpenNeuro support. You said you believed we should not assume that participants are listed in order.

@effigies
Copy link
Collaborator

Okay, here's the situation:

  1. This dataset does not validate with the current validator:

image

  1. The file format uses \r instead of \n newlines, which is just weird since OSX came out. Either way, this caused an older version of the validator to treat it as a single header line, which only rated a warning:

Screenshot from 2024-02-28 10-11-31

  1. Fixing up the participants.tsv to use newlines and sub-* still does not show an error, so this issue is a valid one that needs resolving.

  2. It looks like we can reconstruct the subject IDs because they're still stored in the FIF files:

grep 2399 sub-01/meg/sub-01_task-numbersletters_run-1_meg.fif || echo "Fail"
grep: sub-01/meg/sub-01_task-numbersletters_run-1_meg.fif: binary file matchesgrep 3117 sub-01/meg/sub-01_task-numbersletters_run-1_meg.fif || echo "Fail"
Fail

@effigies effigies added the bug label Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants