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

^Minor¸Topup mean rev_b0 error when 3D data #62

Closed
karanphil opened this issue Jul 11, 2022 · 1 comment
Closed

^Minor¸Topup mean rev_b0 error when 3D data #62

karanphil opened this issue Jul 11, 2022 · 1 comment

Comments

@karanphil
Copy link

karanphil commented Jul 11, 2022

There is a small issue with the command scil_image_math.py mean $rev_b0 $rev_b0 --data_type float32 -f in the topup process. If the rev_b0 is in 3D (which happens to be my case quite often), the scilpy script returns an error and tractoflow continues with the unchanged $rev_b0. This is only a minor issue, since all it implies is that the dtype of $rev_b0 remains unchanged (so probably int16 instead of float32). This is a small detail that only affects the Ants command right after, but I suggest to add a scil_image_math.py convert command to be sure that the processes are always with float32.

@arnaudbore
Copy link
Contributor

@karanphil does not seem to be still in the master. Do you still have this issue or can we close it ?

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

2 participants