Hackathon Project 8: Add CUDA-managed projection container support (continuation of the image stage)#1694
Merged
KrisThielemans merged 2 commits intoUCL:masterfrom Mar 14, 2026
Conversation
KrisThielemans
approved these changes
Mar 13, 2026
Contributor
Author
|
Dear Professor @KrisThielemans , |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is PART of Hackathon Project 8: update STIR data to CUDA managed pointers, then adapt sirf.STIR data-containers and expose the underlying CUDA managed pointer to Python. It is a continuation of the earlier image stage of the same task. The Stage 1 STIR image part ("images first", via VoxelsOnCartesianGrid for CUDA-managed images) has already been merged upstream in #1693, while the companion Stage 1 SIRF part remains separate in SyneRBI/SIRF#1380.
This new PR contains the STIR continuation for the next two stages of the project. Stage 2 extends the managed-memory path from images to acquisition/projection data, by adding the ProjDataInMemory constructor/storage seam needed for CUDA-managed projection containers. Stage 3 then uses that same STIR continuation as the repository-level basis for the end-to-end managed-memory reconstruction proof. The companion SIRF work is in the fork under Dimitra-Kyriakopoulou/SIRF: the earlier Stage 1 image branch Hackathon_Project8 and the current continuation branch Hackathon_project8_II. The runnable benchmark/proof repository for the whole Project 8 path, including Stage 1, Stage 2, and Stage 3, is Dimitra-Kyriakopoulou/ hackathon_project_08_stir_cuda_managed_pointers.
Changes in this pull request
Testing performed
companion SIRF/benchmark path:
Related issues
Checklist before requesting a review
documentation/release_XXX.mdhas been updated with any functionality change (if applicable)Contribution Notes
Please tick the following: