search

RTKConsortium / RTK

Reconstruction Toolkit
Apache License 2.0
241 stars 143 forks source link

BUG: Fixes on Boellaard Scatter Filter #547

Open GabrieleBelotti opened 1 year ago

GabrieleBelotti commented 1 year ago

Changed the condition on AirThreshold value (less than AirThreshold now) and revisited the averaging of the "averageBehindPatient" value. Discussion on #454.

SimonRit commented 1 year ago

I think you can modify this PR with only the change on the threshold. The rest should stay I believe and it would be good to have a few documentation notes. If you'd like, I can prepare a new PR later on.

GabrieleBelotti commented 1 year ago

I think you can modify this PR with only the change on the threshold. The rest should stay I believe and it would be good to have a few documentation notes. If you'd like, I can prepare a new PR later on.

Agreed on the documentation notes. I've made the proper corrections. Let me know if I should do anything else. I believe you should be able to squash commits when merging the pull request.