-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reconstruct a subset of time indices #398
Conversation
Codecov Report
@@ Coverage Diff @@
## main #398 +/- ##
========================================
+ Coverage 7.14% 7.16% +0.02%
========================================
Files 25 25
Lines 4435 4436 +1
========================================
+ Hits 317 318 +1
Misses 4118 4118
|
Preview page for your plugin is ready here: |
Should d2331d9 be ported into this PR? |
changed initialization
Great call @ziw-liu. I cherry-picked that commit and modified the condition to create a new array (instead of checking if 0 is in time_indices, I'm checking if an array already exists). |
Co-authored-by: Ziwen Liu <67518483+ziw-liu@users.noreply.github.com>
Co-authored-by: Ziwen Liu <67518483+ziw-liu@users.noreply.github.com>
Thanks for the improvements @ziw-liu! |
* add `time_indices` to config file * `apply-inv-tf` for subsets of timepoints * test single time index, list of indices, and "all" * check invalid times * test time axis lengths * catch off by one errors * save to output indices * formattings * better tests * changed output shape changed initialization * create an emtpy array if one doesn't exist * fix time indexing * fix tests * Update recOrder/cli/apply_inverse_transfer_function.py Co-authored-by: Ziwen Liu <67518483+ziw-liu@users.noreply.github.com> * remove unused lines * Update recOrder/cli/apply_inverse_transfer_function.py Co-authored-by: Ziwen Liu <67518483+ziw-liu@users.noreply.github.com> --------- Co-authored-by: Clinton Huynh <clinton.huynh@czbiohub.org> Co-authored-by: Ziwen Liu <67518483+ziw-liu@users.noreply.github.com>
recorder reconstruct
naively loops through all time points, applying the reconstruction as it goes. This default is limiting for datasets with a large number of time points where it is more convenient to reconstruct a single time point on a single core (via multiprocessing or slurm).This PR adds a configuration-file option
time_indices
with signature:time_indices: Union[NonNegativeInt, List[NonNegativeInt], Literal["all"]] = "all"
By default
time_indices: "all"
reconstructs all time point, which is backwards compatible.If
time_indices: 5
, then only time-point 5 will be reconstructed.Finally, if
then the 0th, 5th, and 10th indices will be reconstructed.
@edyoshikun requested this feature for simplifying parallel reconstructions with
recOrder
, and I expect we'll use a similar feature inmantis
.