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

Niftyreg interpolation choice not respected #477

Open
rijobro opened this issue Nov 20, 2019 · 1 comment
Open

Niftyreg interpolation choice not respected #477

rijobro opened this issue Nov 20, 2019 · 1 comment
Labels
bug external-fix-required someone else needs to fix something

Comments

@rijobro
Copy link
Contributor

rijobro commented Nov 20, 2019

As per KCL-BMEIS/niftyreg#59, my impression is that cubic interpolation is always used during reg_aladin. The bug is therefore present in our SIRF wrappers around it.

This means that resulting images may contain negatives, which could be problematic (e.g., HKEM expects priors with non-negative voxel values).

The short-term fix for this (whether using NiftyReg directly or using the SIRF wrapper) is to register images, extract the transformation matrix and then resample using that matrix. Using niftyreg directly would look something like:

reg_aladin -ref ${source} -flo ${moved} -rigOnly -interp 0 -aff tm_aladin.txt
reg_resample -ref ${source} -flo ${moved} -res ${output} -inter 0 -trans tm_aladin.txt
@rijobro rijobro added bug external-fix-required someone else needs to fix something labels Nov 20, 2019
@rijobro
Copy link
Contributor Author

rijobro commented Nov 20, 2019

@HarryMarquis once this is sorted, it should simplify your workflow that we were setting up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug external-fix-required someone else needs to fix something
Projects
None yet
Development

No branches or pull requests

1 participant