Live reconstruction compatibility features #388
Closed
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.
This PR adds two features to make
recOrder
's CLI reconstruction workflow more compatible with live reconstructions:a new
time_indices
option for reconstructing a subset of all time points.time_indices
can be an integer, a list of integers, orall
(default). This allows the user to reconstruct all time points (typical for offline), or individual time points (typical for faster online reconstructions).support for passing
ReconstructionSettings
objects directly intoapply_inverse_transfer_function_cli
. This feature makes it more convenient for an online reconstruction routine to call the same reconstruction on different time points...instead of modifying and save a configuration yaml file for each time point, the user can modify thetime_indices
attribute of theReconstructionSettings
object directly.@ziw-liu I'm not sure if I implemented (2) in the way that you had in mind...please suggest improvements.
@clinton-huynh please feel free to add or modify this branch.