pynwb
pynwb copied to clipboard
Deprecate ImageMaskSeries
Motivation
Corresponds to https://github.com/NeurodataWithoutBorders/nwb-schema/pull/583
- Waiting on test suite to pass
- Waiting on schema changes to merge to dev
How to test the behavior?
ImageMaskSeries() # should return a ValueError
Checklist
- [ ] Did you update CHANGELOG.md with your changes?
- [x] Have you checked our Contributing document?
- [x] Have you ensured the PR clearly describes the problem and the solution?
- [x] Is your contribution compliant with our coding style? This can be checked running
ruff check . && codespellfrom the source directory. - [x] Have you checked to ensure that there aren't other open Pull Requests for the same change?
- [x] Have you included the relevant issue number using "Fix #XXX" notation where XXX is the issue number? By including "Fix #XXX" you allow GitHub to close issue #XXX when the PR is merged.
Codecov Report
Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
Please upload report for BASE (
nwb_schema_2.8.0@2c44de4). Learn more about missing BASE report.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| src/pynwb/image.py | 50.00% | 0 Missing and 1 partial :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## nwb_schema_2.8.0 #1941 +/- ##
===================================================
Coverage ? 91.88%
===================================================
Files ? 27
Lines ? 2711
Branches ? 707
===================================================
Hits ? 2491
Misses ? 144
Partials ? 76
| Flag | Coverage Δ | |
|---|---|---|
| integration | 72.85% <0.00%> (?) |
|
| unit | 82.84% <50.00%> (?) |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.