movement icon indicating copy to clipboard operation
movement copied to clipboard

Fix and relax bboxes requirements

Open sfmig opened this issue 1 year ago • 2 comments

TODO

  • [x] implement fixes
    • [x] fix for single individual data
    • [x] relax requirement for extracting frame number (just a number pre-padded with 0s)
    • [x] relax requirement for continuous frame numbers (now they only need to be increasing)
  • [x] make existing tests pass
  • [ ] add tests for new functionality
    • fix for single individual
    • relax requirement for extracting frame number
    • relax requirement for continuous frame numbers

Description

What is this PR

  • [ ] Bug fix
  • [ ] Addition of a new feature
  • [ ] Other

Why is this PR needed?

What does this PR do?

References

Please reference any existing issues/PRs that relate to this PR.

How has this PR been tested?

Please explain how any new code has been tested, and how you have ensured that no existing functionality has changed.

Is this a breaking change?

If this PR breaks any existing functionality, please explain how and why.

Does this PR require an update to the documentation?

If any features have changed, or have been added. Please explain how the documentation has been updated.

Checklist:

  • [ ] The code has been tested locally
  • [ ] Tests have been added to cover all new functionality
  • [ ] The documentation has been updated to reflect any changes
  • [ ] The code has been formatted with pre-commit

sfmig avatar Sep 17 '24 15:09 sfmig

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.78%. Comparing base (4de4963) to head (489cba1). Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #313   +/-   ##
=======================================
  Coverage   99.78%   99.78%           
=======================================
  Files          14       14           
  Lines         912      927   +15     
=======================================
+ Hits          910      925   +15     
  Misses          2        2           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Sep 17 '24 17:09 codecov[bot]