sct-pipeline / fmri-segmentation

Repository for the project on automatic spinal cord segmentation based on fMRI EPI data
MIT License
4 stars 1 forks source link

Centralize data internally at a single place until uploaded to OpenNeuro #31

Closed jcohenadad closed 2 months ago

jcohenadad commented 7 months ago

For training purposes, we need to be organized. Until data are uploaded on OpenNeuro (#30), the latest version of the BIDS data needs to be located at: duke:temp/rohan/fmri_sc_seg/datasets

The folder currently looks like this:

image

This is NOT a BIDS dataset that we intend to upload on OpenNeuro (see #30).

TODO:

When training model:

rohanbanerjee commented 7 months ago

All datasets and model weights will be updated here from now on: https://drive.google.com/drive/folders/14rxPz_mWV1AOSULBFFU7A5IT9zX5PvcI?usp=sharing

jcohenadad commented 7 months ago

unclear what's the diff between 'all' and 'included':

image

jcohenadad commented 7 months ago

in : data_included_bids_20240113.zip

the file organization does not follow our latest plan: https://github.com/sct-pipeline/fmri-segmentation/issues/30#issuecomment-1920085610

eg: sub-nwTactile03_task-tactile_bold.nii.gz is a single volume (likely the moco mean)

tagging @MerveKaptan

jcohenadad commented 7 months ago

image

the purpose of putting the data in a temp location is to make sure the data is fine BEFORE uploading to ON, while @rohanbanerjee can still train his models

HOWEVER: the data "data_included_bids_20240113" does NOT include all the data and it SHOULD. If @rohanbanerjee wants to exclude data for training, that exclusion shouldl be defined by a YAML/JSON file, and NOT by excluding the physical data. Otherwise, this data collection will get confusin, we will forget to put data, blabla

jcohenadad commented 5 months ago

i still see:

image

unclear what each ZIP file is-- please add a README @rohanbanerjee

jcohenadad commented 5 months ago

ok-- nvm-- didn't see the README-- in the future README should be where data is--

rohanbanerjee commented 2 months ago

The dataset has been uploaded to Openneuro, the purpose of this issue is solved now.