search

AMP-SCZ / utility

Storehouse for all utility scripts
Apache License 2.0
0 stars 4 forks source link

Update dpimport_mriqc.sh #48

Closed kcho closed 2 years ago

kcho commented 2 years ago

The csv path on Line 37 needs to be inside quotation marks to avoid error.

tashrifbillah commented 2 years ago

It is correct in its current form. What error did you get?

kcho commented 2 years ago

It is correct in its current form. What error did you get?

This is the error message that I get from running dpimport_mriqc.sh in the current main branch.

usage: import.py [-h] [-c CONFIG] [-d DBNAME] [-v] expr
import.py: error: unrecognized arguments: MRI_ROOT/derivatives/quick_qc/sub-YA01508/YA-YA01508-mriqc-day1to1.csv

My PR fixes this.

tashrifbillah commented 2 years ago

Hi Kevin, thank you for contributing. It was actually my mistake. I have " " in other importers already. https://github.com/AMP-SCZ/utility/blob/main/dpimport_eegqc.sh

tashrifbillah commented 2 years ago

Deployed.