add TOF mashing to SSRB
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- C++
- Stars
- 160
- Forks
- 113
- Avg merge
- 12d 15h
- Merged PRs (30d)
- 1
Description
Some pointers to enable this.
The ProjDataInfo code:
- this would need be deleted
https://github.com/UCL/STIR/blob/8ced2d73933420457e0bc76074964b7d4ff00f0c/src/buildblock/SSRB.cxx#L50-L51 - This looks fine
https://github.com/UCL/STIR/blob/8ced2d73933420457e0bc76074964b7d4ff00f0c/src/buildblock/SSRB.cxx#L134-L140
The actual rebinning code:
- compute
num_tof_bins_to_combinelike for views - add
inandout_timing_pos_numin various places when constructing the sinos - loop over
out_timing_pos_numhere - loop over
in_timing_pos_numhere - add
num_tof_bins_to_combinein the normalisation step
https://github.com/UCL/STIR/blob/8ced2d73933420457e0bc76074964b7d4ff00f0c/src/buildblock/SSRB.cxx#L277
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in src/buildblock/SSRB.cxx, reviewing the linked ProjDataInfo sections and the view-rebinning logic around lines 186, 242, 262, and 277. Trace how input and output sinograms are constructed, then follow the listed loops and normalization step. Done means SSRB combines TOF bins consistently through these stages.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- data
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100