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

Default value for write_bounding_box paramter is not defined in pypreprocess/nipype_preproc_spm_utils.py #312

Open
tbng opened this issue Oct 9, 2018 · 0 comments

Comments

@tbng
Copy link

tbng commented Oct 9, 2018

When trying to do preprocessing with Pypreprocess I encountered the error:

Traceback (most recent call last):
  File "preproc.py", line 71, in <module>
    caching=True,
  File "/home/bnguyen/build/dev_env_1/lib/python3.6/site-packages/pypreprocess/nipype_preproc_spm_utils.py", line 1445, in do_subject_preproc
    **kwargs.get("interface_normalize", {}))
  File "/home/bnguyen/build/dev_env_1/lib/python3.6/site-packages/pypreprocess/nipype_preproc_spm_utils.py", line 858, in _do_subject_normalize
    write_bounding_box=write_bounding_box,
NameError: name 'write_bounding_box' is not defined

Apparently in the commit 47b5149#diff-3eb7a5e605c5b2a38cb0f366d23052a6 the default value for the paramter write_bounding_box is removed by @AGrigis . I can create a PR for this but I don't really know what should be the default value of this parameter (the last default value is [[-78, -112, -50], [78, 76, 85]] but has been commented out before this commit.

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

No branches or pull requests

1 participant