hdmf icon indicating copy to clipboard operation
hdmf copied to clipboard

Check DynamicTableRegion data is in bounds

Open rly opened this issue 1 year ago • 1 comments
trafficstars

Motivation

Fix #210

Checklist

  • [x] Did you update CHANGELOG.md with 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?

rly avatar Aug 10 '24 22:08 rly

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.

codecov[bot] avatar Aug 10 '24 22:08 codecov[bot]

#1293 should be merged before this

rly avatar Oct 04 '25 06:10 rly