-
Notifications
You must be signed in to change notification settings - Fork 6
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
mirt-demo on GPU #5
Comments
Funny timing because just yesterday I fixed an issue with LinearMapsAA.jl to enable CUDA there: There are multiple demos in this repo, so please let me know which one you were trying, and I'll look into it. |
I was looking at this demo in particular: mri-sim-2d+t.jl. A few modifications were made to make things up-to-date with deprecations to MIRT (moving to MIRTjim). Updating does not seem to fix this issue, but I think the complexity of the demo makes it hard to debug in itself. I created a smaller example that throws a different error when using GPU, which could be useful to track down bugs when you get the time.
|
@pkash16 FYI I am working on this... Currently I am waiting for an update to NFFT.jl for CUDA. |
Not sure if this is an issue with my implementation, this demo, MIRT.jl, or LinearMapsAA.jl, so I'm putting this issue here. Let me know if there is a better place for this.
It seems like from this discussion #JeffFessler/MIRT.jl#105 the goal is to have MIRT.jl "just work" with the GPU as everything is implemented with AbstractArray types. I am running this mirt-demo for "temporal finite differences" reconstruction of dynamic MRI data.
To this demo I added these lines:
Constructing the A matrix receives no errors. But doing the multiplication on the next line
A * xtrue
does create the following error message "This object is not a GPU array", which seems to stem from an unhappy multiplication in the LinearMapsAA package. The error message is hard for me to parse so if you happen to be familiar with this and have any hints, I am happy to dig deeper and figure out what is going on since I have a GPU set up on my end, but currently am stumped parsing this error message.p.s. we have a similar reconstruction at USC for dynamic speech using acquired spiral sampled data in MATLAB + GPU, and I am very curious to see how Julia + GPU performs hence the interest in this demo!
The text was updated successfully, but these errors were encountered: