Skip to content

Add detections validator#117

Merged
sfmig merged 13 commits into
mainfrom
smg/detections-ds-validator
Nov 20, 2025
Merged

Add detections validator#117
sfmig merged 13 commits into
mainfrom
smg/detections-ds-validator

Conversation

@sfmig
Copy link
Copy Markdown
Member

@sfmig sfmig commented Nov 19, 2025

Description

This PR:

  • adds a validator for a bounding box detections dataset
  • adds an abstract base class for valid ethology datasets
  • factors out the validators module
  • extends the dataset validators (for bbox annotations and detections) to check a minimum set of dimensions per data variable
  • adds tests

Also:

  • fixes to sphinx docs to prevent warnings when parent classes
  • renaming of fixtures related to ethology datasets for consistency and clarity

References

Related to PR #114

How has this PR been tested?

Tests pass locally and in CI

Is this a breaking change?

No.

Does this PR require an update to the documentation?

Included.

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

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.45%. Comparing base (48cdd82) to head (10fc95b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
+ Coverage   99.42%   99.45%   +0.02%     
==========================================
  Files           6        8       +2     
  Lines         525      553      +28     
==========================================
+ Hits          522      550      +28     
  Misses          3        3              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sfmig sfmig marked this pull request as ready for review November 20, 2025 20:22
@sfmig sfmig merged commit 446dde4 into main Nov 20, 2025
16 checks passed
@sfmig sfmig deleted the smg/detections-ds-validator branch November 20, 2025 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant