1 #1
Merged
1 #1
Conversation
This file contains 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
Needed some changes in ProjDataGEHDF5 to construct the projdata_info itself
- fixed some types to be consistent - commented out unused m_NX etc
some code wasn't used, and generated warnings with clang.
…ic for checking Signa
Co-authored-by: Kris Thielemans <KrisThielemans@users.noreply.github.com>
- Scanner.cxx name has to match with what GE uses - we were initialising a local scanner_sptr as opposed to the member variable
…canner Otherwise we get conflicts on the scanner
write some warnings and reading effective-ring-radius
related #555, fixes evguenis bug report.
also correct number of rings for DMI3
Multiplicative factor taking duration into account was wrong (for listmode data we were using times duration as opposed to divide by it, for sinogram data it was 1). In addition, we also take decay into account now (assuming F-18).
construct_randoms_from_GEsingles needs to be called with the actual unlisted projdata. Otherwise, timing information would be incorrect.
do avoid some duplication, I've moved the norm stuff to a separate script
InvertAxis cleanup. Fixes #459
Also moved printing of exam_info to ExamInfo::parameter_info() to reduce duplication
extend GE RDF9 raw data support to other GE PET/CT scanners that use RDF9
added list_lm_info
Release4 to master 16 sep2020
…the epsilon term in the denominator square.
Add GE ct_slopes
there was a missing }, and GE values for kev/kvp were wrong
Fix an issue with the gradient of the relative difference prior (also change default values)
Add input-function file checks
* Performing SSRB with a template sinogram file * Renamed `span` to `num_segments_to_combine` Co-authored-by: Kris Thielemans <KrisThielemans@users.noreply.github.com>
Release_4 to master 10oct2020
The Doxygen doc wasn't completely updated in #368
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.
test