Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better ini error message when ini file is not in the run folder #221

Merged
merged 1 commit into from
May 27, 2016

Conversation

arthurmensch
Copy link
Contributor

@arthurmensch arthurmensch commented May 27, 2016

All in the title.

The modified file is not tested (which is bad), but much bravoure would be needed to address this terrible lack.

running python examples/easy_start/nipype_preproc_spm_auditory.py from root now yields:

Traceback (most recent call last):
  File "examples/easy_start/nipype_preproc_spm_auditory.py", line 49, in <module>
    subject_data = do_subjects_preproc(jobfile, dataset_dir=dataset_dir)[0]
  File "/home/arthur/.conda/envs/py27/lib/python2.7/site-packages/pypreprocess/nipype_preproc_spm_utils.py", line 1529, in do_subjects_preproc
    ' as the .ini file' % subject_factory)
IOError: Could not load spm_auditory_preproc.ini, please make sure to run your script within the same folder as the .ini file

@bthirion
Copy link
Member

LGTM.

@dohmatob
Copy link
Contributor

LGTM2, merging.

@dohmatob dohmatob merged commit 13f8c67 into neurospin:master May 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants