tvb-ukbb
tvb-ukbb copied to clipboard
inconsistent variable names, script names & hard-coded things
Things to clean up:
- [ ] subject directory variable naming conventions (e.g., $subjdir, $direc, $baseDir are all used to mean the same directory)
- [ ] turn off verbose mode in
bb_eddy_wrapandbb_probtrackx2 - [x] rename TVB team scripts with 'tvb_' prefix (e.g., probtrackx scripts)
- [ ] rename IDP pipeline
bb_IDP.pyto be consistent with other subpipes
Hard-coded things to fix:
- [ ] B0 extraction in diffusion subpipe << This is an issue for the odd case where B0 isn't the first volume
- [ ] acqparams.txt contents in
bb_pre_eddy
Also some typos that don't affect pipeline function:
- [ ] extra subject name argument in FEAT job submission in
bb_pipeline_funcin both ADNI3 and master branches
Also need to check for consistent labelling of QC report with pipeline sections (e.g., 'dMRI' not 'DTI', 'structural' not 'anatomical') @justinliuyuwang
And make logging of jobFC equivalent to other logs (missing header and footer?)