Open MDijsselhof opened 1 year ago
Please see the attached files for my recommendation for the next ExploreASL BIDS output version of participants.tsv, accompanied by the BIDS participants.json (which I could only upload as .txt).
Please see the attached files for my recommendation for the next ExploreASL BIDS output version of participants.tsv, accompanied by the BIDS participants.json (which I could only upload as .txt).
Did you check the BIDS compliance? E.g., I wouldn't add the xASL subject ID, and session_id is missing. I would first check how BIDS does this. The participants.json looks nice!
Comment to myself: add disease type column as well
Please see the attached files for my recommendation for the next ExploreASL BIDS output version of participants.tsv, accompanied by the BIDS participants.json (which I could only upload as .txt). participants_CBA.tsv.csv participants.txt
Did you check the BIDS compliance? E.g., I wouldn't add the xASL subject ID, and session_id is missing. I would first check how BIDS does this. The participants.json looks nice!
xASLsubject ID I've kept so Candace can identify the subjects we have now, just as an example. session_id is missing, but its partly in the participant_id and I haven't found any example that uses that (also its not required). The link for the wiki.
Agreed?
Updated version for Candace:
participants.txt participants_CBA.tsv.csv
@HenkMutsaerts xASLsubject ID I've kept so Candace can identify the subjects we have now, just as an example. session_id is missing, but its partly in the participant_id and I haven't found any example that uses that (also its not required). The link for the wiki.
Agreed?
Description
To streamline CVAge pre-processing, ExploreASL could provide output .csv's and the participants.tsv in a different manner. Requirements are:
[ ] Note that
participants.tsv
inrawdata
is not copied to derivatives, this needs fixing: -> add it to copy_to_derivative.m -> xASL_bids_CompleteBIDS2Legacy->104 xASL_bids_FinishImport_CreateParticipants -> here we copy participants.tsv from the root folder, but we should (also) copy it from the rawdata folder (per BIDS) This is addressed here: #1658[ ] Combine all ExploreASL output (specifically CBF but also later on CoV, Tex, ATT etc.) in participants.tsv
[ ] Male/Female input should always be M/F per BIDS
[ ] WMH output should be NaN if not available, or 0 if none found, see link to issue: https://github.com/ExploreASL/ExploreASL/issues/1486
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