hdmf
hdmf copied to clipboard
Check DynamicTableRegion data is in bounds
trafficstars
Motivation
Fix #210
Checklist
- [x] Did you update
CHANGELOG.mdwith your changes? - [x] Does the PR clearly describe the problem and the solution?
- [x] Have you reviewed our Contributing Guide?
- [x] Does the PR use "Fix #XXX" notation to tell GitHub to close the relevant issue numbered XXX when the PR is merged?
Codecov Report
:x: Patch coverage is 16.12903% with 26 lines in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 37.91%. Comparing base (886bf4a) to head (fa23969).
| Files with missing lines | Patch % | Lines |
|---|---|---|
| src/hdmf/common/table.py | 16.00% | 21 Missing :warning: |
| src/hdmf/container.py | 16.66% | 5 Missing :warning: |
:exclamation: There is a different number of reports uploaded between BASE (886bf4a) and HEAD (fa23969). Click for more details.
HEAD has 4 uploads less than BASE
Flag BASE (886bf4a) HEAD (fa23969) 5 1
Additional details and impacted files
@@ Coverage Diff @@
## dev #1168 +/- ##
===========================================
- Coverage 91.62% 37.91% -53.72%
===========================================
Files 42 42
Lines 9684 9707 +23
Branches 1963 1971 +8
===========================================
- Hits 8873 3680 -5193
- Misses 523 5696 +5173
- Partials 288 331 +43
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
#1293 should be merged before this