Skip to content

antsRegistrationSyNQuick- spelling error? #2543

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.

Sign up for GitHub

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

Jump to bottom
Closed
atsuch opened this issue Apr 16, 2018 · 3 comments
Closed

antsRegistrationSyNQuick- spelling error? #2543

atsuch opened this issue Apr 16, 2018 · 3 comments
Labels
bug
Milestone
1.0.3

Comments

@atsuch
Copy link

atsuch commented Apr 16, 2018

Summary

I'm not sure why others wouldn't have caught this one, but I think there is a spelling mistake in the command for antsRegistrationSyNQuick.sh...

I think it has to be the capital N for SyN, but in nipype/interfaces/ants/registration.py the _cmd is antsRegistrationSynQuick.sh

Actual behavior

Causes a crash when using a node with antsRegistrationSyNQuick with the following message

IOError: No command "antsRegistrationSynQuick.sh" found on host c2. Please check that the corresponding package is installed.

Platform details:

I am using nipype 1.0.2, ants version 2.1.
@effigies
Copy link
Member

Yup, you're right. Any interest in submitting a fix? The _cmd attribute and test commands in the doc string will need updating.

@effigies effigies added the bug label Apr 16, 2018
@effigies effigies added this to the 1.0.3 milestone Apr 16, 2018
@effigies effigies mentioned this issue Apr 17, 2018
Merged
@effigies
Copy link
Member

@atsuch This will be fixed in 1.0.3.

@atsuch
Copy link
Author

atsuch commented Apr 19, 2018

@effigies, Thank you!

I will try contributing next time...!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
1.0.3
Development

No branches or pull requests
2 participants
@effigies @atsuch