Add gaussian and poisson noise options to projection applications#860
Add gaussian and poisson noise options to projection applications#860axel-grc wants to merge 1 commit intoRTKConsortium:mainfrom
Conversation
axel-grc
force-pushed
the
Noise
branch
2 times, most recently
from
b2d59a8 to
ef55545
Compare
axel-grc
force-pushed
the
Noise
branch
3 times, most recently
from
aee5dac to
1cbeb80
Compare
lesaintjerome
left a comment
lesaintjerome
left a comment
There was a problem hiding this comment.
I did a few tests. It seems to run fine.
The noise feature in rtkdrawgeometricphantom and rtkprojectshepplogan has not been removed. Should it be?
| section "Gaussian noise" | ||
| option "gaussian" - "Gaussian noise parameters: <mean> Noise level and <std> Noise standard deviation" double multiple no | ||
|
|
||
| section "Poisson noise" | ||
| option "poisson" - "Poisson noise parameters: <I0> Number of impinging photons per pixel and <muref> reference linear attenuation coefficient" double multiple no |
There was a problem hiding this comment.
This seems to make the mu_ref parameter mandatory. Why not. Though we could mention something like "e.g. set 0.01879, which is the attenuation coefficient of water at 75 keV".
There was a problem hiding this comment.
Same could be said about the mean parameter for gaussian noise. Default=0. would make sense, to me.
There was a problem hiding this comment.
The problem is that we can't use different default values for lists with gengetopt, so the same default value will be used for mean and std for example.
We can maybe add theses default values to the implementation part ?
There was a problem hiding this comment.
Or have slightly more complex Gaussian and Poisson ggo sections, with 2 parameters each?
axel-grc
changed the title
3 participants
No description provided.