-
Notifications
You must be signed in to change notification settings - Fork 7
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
VolumetricImagingExtractor #248
Conversation
I will recruit @alessandratrapani to help review this PR as well - maybe we'll go over it at the retreat? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, just a minor correction
Co-authored-by: Alessandra Trapani <55453048+alessandratrapani@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## main #248 +/- ##
==========================================
+ Coverage 76.98% 77.55% +0.56%
==========================================
Files 37 38 +1
Lines 2724 2793 +69
==========================================
+ Hits 2097 2166 +69
Misses 627 627
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Fixes #245
See also discussion in #241