Skip to content

hotfixed device error in interpolation#84

Open
J-Hizzle wants to merge 2 commits intoXzzX:mainfrom
J-Hizzle:main
Open

hotfixed device error in interpolation#84
J-Hizzle wants to merge 2 commits intoXzzX:mainfrom
J-Hizzle:main

Conversation

@J-Hizzle
Copy link
Contributor

@J-Hizzle J-Hizzle commented Mar 3, 2026

When I tried to compile on a GPU system, there was an error in interpolation.cpp due to the host trying to access information stored in device memory. I fixed it and now MRMD is GPU safe again.

@XzzX
Copy link
Owner

XzzX commented Mar 4, 2026

Hmm, unfortunate, actually running ctest in the CI and not just compiling would have found this. But GitHub does not have GPU runners available in their free plan...

@XzzX
Copy link
Owner

XzzX commented Mar 4, 2026

The change is pretty expensive. You are copying to host only to get some boundaries. Does the output have to be a histogram? If a ScalarView is enough you can avoid all the copying.

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

Successfully merging this pull request may close these issues.

2 participants