tkkarjal / magia

Automated brain PET analysis pipeline
https://doi.org/10.3389/fninf.2020.00003
MIT License
27 stars 7 forks source link

Change hard-coded paths to getenv() for pipeline portability #30

Closed eebrown closed 5 years ago

eebrown commented 5 years ago

To help run the pipeline on other systems, I replaced hard-coded paths with environment variables. The following new environment variables must be set (with the formerly hard-coded directories as example):

setenv('ARCHIVE_DIR', '~/vsshp.net/Research/Neurotiede/aivo/archive');
setenv('FS_FINAL_DIR', '~/vsshp.net/Research/Neurotiede/aivo/FreeSurfer');
setenv('FS_TEMP_DIR', '/scratch/shared/megapet/FreeSurfer_temp');
setenv('SPM_DIR', '/scratch/shared/toolbox/spm12'); 

In addition to the environment variables previously required (DATA_DIR, MRI_DIR, MAGIA_PATH, MAGIA_ARCHIVE).

tkkarjal commented 5 years ago

Looks good, thanks! Merged.