adding different distance relationship for collimator response modelling for SPECT
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 160
- Forks
- 113
- Avg merge
- 12d 15h
- Merged PRs (30d)
- 1
Description
STIR currently uses sigma = a d + b, while many other people use sigma = sqrt(sigma_intr^2 + (d a)^2). It'd be nice to support the latter as well. Obviously, people will have to check which one fits their scanner/collimator best.
Necessary code modifications seem quite small. The actual code to generate sigma is
https://github.com/UCL/STIR/blob/e2f0bcb74af6bbf067785e07f58a8430faf3c64d/src/recon_buildblock/SPECTUB_Tools.cxx#L540 (note: do_fb is for fan-beam collimators)
- we'd need an extra field
COL.do_parallel_with_sqrt. could reuse theAandBfields to make things confusing, or add 2 new ones. I don't mind which really as this is internal code.
` - we'd need to add members/methods and keywords for ProjMatrixByBinSPECTUB, e.g. at
https://github.com/UCL/STIR/blob/e2f0bcb74af6bbf067785e07f58a8430faf3c64d/src/recon_buildblock/ProjMatrixByBinSPECTUB.cxx#L103-L104
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 at src/recon_buildblock/SPECTUB_Tools.cxx around line 540, where sigma is generated, and inspect the do_fb handling. Then read ProjMatrixByBinSPECTUB.cxx around lines 103-104 for the related members, methods, and keywords. Done means the alternate distance relationship can be configured and used for SPECT collimator response modelling.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- computer-vision
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100