nested Patlak and generalized Patlak OSMAPOSL (POSMAPOSL & GPOSMAPOSL) and nested Richardson-Lucy Motion Deconvolution Image Reconstruction (RLMC 10BC0 IR)#1685
Open
nkarakatsanis wants to merge 3445 commits intoUCL:masterfrom
Conversation
added 30 commits
February 13, 2012 17:08
- many updates and clarifications, especially related to using CMake
… if all options were given (i.e. if you attempted to use the defaults, the program crashed). Also, confusingly it printed text with ACF_threshold once it was going, while the command line option is ACF-threshold.
…ite (otherwise it doesn't generate the documentation)
…of using a const member as size for an array (needed for VC2010)
…y uses functions from those
…ing about WIN32 on cygwin is still there. - add check on system getopt here (not only in buildblock) as stir/getopt.h need to know
…/getopt.h needs to know always
…re we called error()) - slightly better error message when reading the matrix failed
- add a few more classes
…sed to private typedef (as this confuses SWIG). This change makes sense anyway.
…pedef (as this confuses SWIG). This change makes sense anyway.
- adapt ITKImageInputFileFormat::read_from_file to check if it's a DICOM file, and if so, attempt to read the whole volume (differentiating on series/frame/gate)
…ectory info was not given
…aw_ECAT_projdata_to_interfile.cxx
The script checks scatter against a previous result. However, there's a random number generator is used to assign the exact location of the scatter points. With the current settings, the difference can be larger than the previous threshold, so I increased it to 4% of the max.
- ECAT7 projection data was read incorrectly (wrong order of arguments in a function) - create_projdata_template sets modality to PET
- handle filesnames with multiple dots - have sensible voxel-sizes for tangential and axial dimensions
It is highly recommended to use these as opposed to fwdtest and bcktest in your scripts. Note the different order of the arguments!
- removed a few hard-wired filenames in simulate_data.sh (and use its arguments properly) - fixed creation of randoms such that the same command would be usable for zero randoms
- moved scripts and *txt files one level up - moved samples, examples and samples/PET_simulation to new sub-dir examples in the top-level directory - created CMakeLists.txt in top-level directory (and moved PROJECT and other main statements there from src/CMakeLists.txt). - moved CMake helper files to src/cmake, adjust many CMakeLists.txt to rely on module path for these files (by no longer specifying location using paths)
…version warnings though)
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Changes in this pull request
This PR introduces nested Patlak and generalized Patlak OSMAPOSL (POSMAPOSL & GPOSMAPOSL) reconstruction.
In addition, nested Richardson-Lucy Motion Deconvolution Image Reconstruction (RLMCIR) is featured, too.
Testing performed
Related issues
Checklist before requesting a review
documentation/release_XXX.mdhas been updated with any functionality change (if applicable)Contribution Notes
Please tick the following: