Add MockSegmentationInterface
#1067
Conversation
| @@ -162,24 +165,52 @@ class MockImagingInterface(BaseImagingExtractorInterface): | |||
| A mock imaging interface for testing purposes. | |||
| """ | |||
|
|
|||
| ExtractorModuleName = "roiextractors.testing" | |||
There was a problem hiding this comment.
What do these changes have to do with a MockSegmentationInterface?
There was a problem hiding this comment.
This is related to make the MockImagingInterface to work with some of the magic in the tests mixings. I can make this changes in a different PR if that is what you are asking.
There was a problem hiding this comment.
Yeah, I think this is a good idea anyways. Check:
| The sampling frequency of the mock imaging data in Hz, by default 30. | ||
| dtype : str, optional | ||
| The data type of the generated imaging data (e.g., 'uint16'), by default 'uint16'. | ||
| verbose : bool, optional |
There was a problem hiding this comment.
One of these verbose's is redundant.
Co-authored-by: Paul Adkisson <[email protected]>
Co-authored-by: Paul Adkisson <[email protected]>
h-mayorquin
changed the base branch from
main
to
make_get_extractor_work_with_mock_imaging
h-mayorquin
changed the base branch from
make_get_extractor_work_with_mock_imaging
to
main
|
OK, did the changes, #1076 should come before. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1067 +/- ##
==========================================
+ Coverage 90.40% 90.42% +0.01%
==========================================
Files 129 129
Lines 8029 8041 +12
==========================================
+ Hits 7259 7271 +12
Misses 770 770
Flags with carried forward coverage won't be shown. Click here to find out more.
|
As in the title.