Skip to content

fix(detection): don't default tiny80.sod to face-only architecture - #566

Merged
matteius merged 1 commit into
opensensor:mainfrom
davlaw:fix/tiny80-sod-architecture
Aug 26, 2026
Merged

fix(detection): don't default tiny80.sod to face-only architecture#566
matteius merged 1 commit into
opensensor:mainfrom
davlaw:fix/tiny80-sod-architecture

Conversation

@davlaw

@davlaw davlaw commented Aug 26, 2026

Copy link
Copy Markdown

Unrecognized .sod filenames fell through to the :face architecture
fallback, so the tiny80 COCO-80 object model was silently running as
a face-only detector. Adds an exact-match case routing tiny80.sod to
:tiny80, mirroring the existing :voc exact-match case.

Test plan

  • Confirmed via journalctl -u lightnvr that tiny80.sod now logs
    Detected COCO-80 model by exact filename match, using :tiny80 architecture
    instead of falling back to face detection, across all streams using this model.

Unrecognized .sod filenames fell through to the :face architecture
fallback, so the tiny80 COCO-80 object model was silently running as
a face-only detector. Add an exact-match case routing tiny80.sod to
:tiny80.
@matteius

Copy link
Copy Markdown
Contributor

Hey, @davlaw which sod model(s) do you have/use?

@matteius
matteius merged commit b0e1664 into opensensor:main Aug 26, 2026
1 check passed
@davlaw

davlaw commented Aug 26, 2026

Copy link
Copy Markdown
Author

I'm using SOD's Tiny COCO model (tiny80.sod, the 80-class COCO variant, ~65MB) from pixlab.io. This PR came out of debugging why it was silently falling back to the face-only :face architecture — load_sod_model() had an exact-match case for :voc but nothing for tiny80.sod, so it hit the generic unrecognized-.sod-file fallback instead of loading it as :tiny80.

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.

2 participants