Skip to content

Stable ANTs (2.1.0) not working with stable nipype (0.10.0) #1100

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

Closed
gerddie opened this issue May 7, 2015 · 5 comments
Closed

Stable ANTs (2.1.0) not working with stable nipype (0.10.0) #1100

gerddie opened this issue May 7, 2015 · 5 comments

Comments

@gerddie
Copy link

gerddie commented May 7, 2015

The latest release of ANTs broke the nipype registration interface in the stable release. @oesteban told me that there is already a pull request for the nipype developers version, apparently #999, but this only fixes an issues with the flags, and this is not all:

My problem is that the interface is looking for some output_0InverseWarped.nii.gz that was created by ANTS 2.1.0-rc2 +git: 9103999 (the current version in Debian Jessie), but the new ANTs version creates some different .h5 files as composite output transform.

It would be nice if someone could provide a patch against 0.10.0 (or create a 0.10.1 release) that fixes these incompatibilities, so that we can add it to Neurodebian (that already provides ANTs 2.1.0). Packaging the (moving) developer version is unfortunately not an option.

many thanks,
Gert

@satra
Copy link
Member

satra commented May 7, 2015

@gerddie - how about we cut a 0.11 release? we have plenty of updates anyway.

unfortunately, because ants hasn't been versioned consistently (now they are), we cannot use versioning to switch between interfaces.

@gerddie
Copy link
Author

gerddie commented May 7, 2015

@satra - depends on how much changed, i.e. how backward compatible would this new release be?

@gerddie
Copy link
Author

gerddie commented Jul 1, 2015

Just a ping to ask if anything is happening with this problem.

As pointed out above, the problem is specifically introduced if the ANTs version and nipype are installed from the neurodebien repositories. Since Debian package dependencies can be quite easily defined, a patch that fixes this issue could be applied only within the python-nipype package and dependencies could be set that that right version of ANTs is installed from the repo. Hence an ANTs version dependent switch between Interfaces would not be necessary.

many thanks,
Gert

@satra
Copy link
Member

satra commented Jul 1, 2015

@gerddie - versioning is being added to ants - but this will not be backwards compatible. regarding nipype support for the existing ants in neurodebian, would it be possible to try the current master against the existing ants in neurodebian? if that works, we can take the changes in the ants tree and patch against 0.10.0

@chrisfilo - time to release 0.11 ?

@blakedewey
Copy link
Contributor

I have a PR that corrects at one small difference with the new ANTS versions (different CLI option for N4BiasFieldCorrection) that should be merged before this goes through to help compatibility. Unfortunately, I am rebuilding my test distro and cannot make the necessary changes myself. See #1096.

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

4 participants