Skip to content

feat: load OCTRON bounding box CSVs (#1108) - #1139

Open
anshurajbisoyi98-ctrl wants to merge 3 commits into
neuroinformatics-unit:mainfrom
anshurajbisoyi98-ctrl:feature-1108-octron-bboxes
Open

anshurajbisoyi98-ctrl wants to merge 3 commits into
neuroinformatics-unit:mainfrom
anshurajbisoyi98-ctrl:feature-1108-octron-bboxes

Conversation

@anshurajbisoyi98-ctrl

@anshurajbisoyi98-ctrl anshurajbisoyi98-ctrl commented Oct 4, 2026 •

Copy link
Copy Markdown

Fixes #1108

OCTRON exports one CSV per tracked ID, with video metadata at the top of each file. This adds an OCTRON loader to the existing bounding-box loading interface and validates the metadata and tracking columns before constructing a dataset.

A single CSV loads its track. To combine tracks from the same video, pass their files with additional_files; the loader checks that the video name, frame counts, and dimensions match, and rejects duplicate frame/track observations. It preserves the original zero-based frame indices and missing detections, derives box centres and widths/heights from the corner coordinates, and uses OCTRON track IDs as individual names.

The user guide now documents OCTRON’s format and both loading patterns. The explicit additional_files interface is included in this draft for maintainer feedback.

Validation

  • OCTRON and related loader/validator tests: 203 passed, 4 deselected.
  • All pre-commit hooks passed.
  • Sphinx HTML build passed with gallery execution disabled. The default gallery build timed out while fetching unrelated sample video data.
  • GitHub Actions Tests and Docs runs currently require maintainer approval; no remote test result is available yet.

AI assistance: Codex assisted with implementation, regression tests, documentation, and verification.

@anshurajbisoyi98-ctrl
anshurajbisoyi98-ctrl marked this pull request as ready for review October 7, 2026 19:20
@sonarqubecloud

sonarqubecloud Bot commented Oct 7, 2026

Copy link
Copy Markdown

This branch has not been deployed

No deployments
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.

Add support for loading OCTRON bboxes into movement

1 participant