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

eddy cmdline choice #50

Open
yibeichan opened this issue May 10, 2023 · 4 comments
Open

eddy cmdline choice #50

yibeichan opened this issue May 10, 2023 · 4 comments

Comments

@yibeichan
Copy link
Collaborator

there are two cmdline choices in eddy, either eddy_openmp or eddy_cuda, which depend on the input use_cuda. @ghisvail, in your example here, you only used eddy. I also tried use eddy only when converting from nipyper to pydra with our converter, but it doesn't work. Do you have any ideas about how we should deal with eddy_openmp and eddy_cuda? Thank you!

@yibeichan
Copy link
Collaborator Author

never mind, I think eddy works; Dorota and I just found out that my test data has some problems, not the problem from cmdline

@ghisvail
Copy link
Contributor

Yes, it's a complicated story 😅

Prior to the new conda distribution of FSL, there used to be eddy, eddy_openmp and eddy_cuda${CUDA_VERSION}. With eddy becoming deprecated at some point.

With the new conda distribution, eddy_openmp was renamed to eddy_cpu, eddy_cuda${CUDA_VERSION} is still the same, and eddy is now the recommended entypoint which just dispatches to the CPU or CUDA version based on some runtime checks.

@ghisvail
Copy link
Contributor

I focused on the new Conda version (version 6.0.6 onwards) hence the difference with the nipype interface.

@yibeichan
Copy link
Collaborator Author

okay, sounds good. I'm testing the code with some example data (I asked chatgpt to generate the whole set of test files) but got errors with reading epi_index/bvecs.scheme. since I can't push my code because of this issue #51 so I don't actually know whether it's something wrong with the code (I can't show you either) or with the example data. I'll fix the data quota first then come back to this one

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