-
Notifications
You must be signed in to change notification settings - Fork 1
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
CUDArt doesn't work with Julia 0.6 #76
Comments
In case it's useful, here's the segfault I get with the current releases of CUDArt, CUDAdrv, and CUDAapi on 0.6 (I also tried various combinations of master branches and sometimes received different errors)
|
We could take option 3 a step further and use the CuArrays package to implement the mismatch calculation directly. This currently requires a small workaround for our GPUs (see https://github.com/JuliaGPU/CUDAnative.jl/issues/165). If we could use CuArrays without losing efficiency I think it would be a big win. Then we should be able to use nearly the same code for CPU and GPU mismatch calculation. |
I finally updated our code to work with newer versions of CUDArt and CUDAdrv on Julia 0.6. My work is in the branch https://github.com/HolyLab/BlockRegistration/tree/cjg/fix_cuda. I think this would be a good base on which to build 0.7 support, but currently we can't merge with master because I had to remove a Femtocleaner commit that introduced a bug, see #89. Note that one unrelated test still fails as described in #88. In a separate branch |
We can re-run femtocleaner, so I'm fine with bypassing that commit. @kdw503 is starting with smaller repositories than this one, so it may be a while before we get here anyway. |
After a moderate amount of effort I still can't get CUDArt to work with Julia 0.6. It still works on 0.5 on cannon with an out-of-date set of packages. The problems seem to stem from CUDArt's dependence on CUDAdrv and CUDAapi. The latter packages have evolved a lot but according to the README CUDArt is no longer actively developed. Also see the last comments here. (We use the
standalone
branch of CUDArt in cannon's 0.5 packages, which doesn't require those packages). I can see three options for getting this working on 0.6. @timholy which do you think is best?standalone
(requires updating that branch for 0.6 and also updating BlockRegistration)My feeling is that option 3 is the best practice but may be a lot of work.
The text was updated successfully, but these errors were encountered: