sct-pipeline / balgrist-sci

Pipeline for processing SCI patients using SCT
MIT License
2 stars 0 forks source link

Add `process_data.sh` #5

Closed valosekj closed 1 month ago

valosekj commented 2 months ago

This PR adds the process_data.sh script. This script is intended to be the main script used within this repository.

Example usage:

# Clone this repo and go to it
# Then, run:
./process_data.sh \
-d ~/experiments/balgrist-sci/source_data/dir_20231010 \
-b ~/experiments/balgrist-sci/bids \
-r ~/experiments/balgrist-sci/data_processed \
-p sub-001 \
-s ses-01 \
-c T2w dwi \
-a 30 \
-x M

The process_data.sh script does the following steps:

valosekj commented 1 month ago

@sandrinebedard, I think the PR is ready for review. Maybe you could test it on your new laptop to simulate the scenario that you are starting with no libraries installed :-D

sandrinebedard commented 1 month ago

I will test this on my new computer today!

valosekj commented 1 month ago

I am thinking that if the person wants to rerun the analysis, a reuse the already corrected manual seg, he won't be able since the dicom conversion needs to be done every time?

Good point! I've also been thinking about this scenario. I added an extra question asking the user whether to overwrite the folder in case it already exists in https://github.com/sct-pipeline/balgrist-sci/pull/5/commits/9330718390b13c0accd62c8dd5931cea81e91db8.

valosekj commented 1 month ago

Merging after an in-person discussion with @sandrinebedard.