Feature requests & bugfixes: bl-dataset-bidsupload.js

[kathrynalpert]

Feature requests:

• handle sessions (longitudinal data) - need another level of nested dirs, e.g.: sub-subject/ses-session/anat/sub-sub_ses-session_t1w.nii.gz
• allow links (currently, source data has to be a file not a link) - this might require a feature request or PR to node-archiver to add the -h (--dereference) option to tar

Bugfixes:

• line 67 should read from participants.tsv
• line 69 should look for "participant_id" column

[francopestilli]

@kathrynalpert thanks! @stevengeeky

[soichih]

I've added support for sessions, and fixed those bugs. I haven't updated to allow links, however.

@kathrynalpert What files are linked exactly? The directories, or the files? Or maybe both? Are they symlinks or hardlinks?

[kathrynalpert]

[Sorry for my delayed reply - I wasn't receiving notifications, but should be now.]

@soichih In my case, the links are symlinks to files, and it seems that your new code (1.2.38) works with them!

[kathrynalpert]

One comment about session handling: The upload works beautifully and the main "datasets" tab has the sessions listed nicely. However, when I'm staging datasets for processing, only the subject ID is visible (see screenshot, I'm talking about how to distinguish between last two datasets), so I have to use timestamp to determine which session I'm selecting. It'd be great if the session were visible here, too (or even better, if I could filter by session).

[soichih]

@kathrynalpert Thanks for the feature request. I've added session labels on the dataset selecter and a few other places. Please let me know if you notice anywhere else that I should display it. By the way, you can use this repo for posting Brainlife UI issue / requests > https://github.com/brain-life/warehouse/issues

[kathrynalpert]

Fabulous changes! I'm closing this issue and will make UI requests there in future.