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

Replacing SPM, FSL, REISERT tools with standard Python packages #59

Open
tashrifbillah opened this issue Feb 28, 2020 · 1 comment
Open
Labels

Comments

@tashrifbillah
Copy link
Collaborator

An experiment is in progress to minimize external dependencies of harmonization software. For this purpose, py-devel branch has been created that comes with three environment variables:

DTIFIT_TOOL={FSL,DIPY}
INTERP_TOOL={SPM,SCIPY}
UNRING_TOOL={REISERT,DIPY}

spm-bspline (default) branch has the first value from respective sets. On the other hand, second value in each set corresponds to standard Python packages. In py-devel branch, @tashrifbillah set the second value for all the variables and obtained good results on his custom-made connect_prisma test data. Notably, the custom-made test data does not even contain full brain and contain only 15 gradients for each DWI. Since the algorithm performs well on this so-called test data, it should have the potential of performing well on real DWI data.

This experiment opens scope for replacing FSL, SPM, and REISERT dependencies with standard Python libraries while preserving functionality of the software. Convinced by the outcome of this experiment, we can comment that maybe travelHeads=0/1 was the only bug that yielded discrepancies between Python and MATLAB software. The above standard libraries may have been performing well but we were not looking at the right cause and so ended up replacing them in first place.
cc @sbouix @yrathi @suheyla2

@tashrifbillah
Copy link
Collaborator Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant