Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup Detector labelmap #4932

Merged
merged 2 commits into from
Jan 6, 2023

Conversation

NateMeyer
Copy link
Contributor

  1. Fixup OpenVino labelmap to use Car instead of Truck
  2. Add missing indexes to default labelmap.txt
  3. Remove TensorRT-specific labelmap, use default.
  4. Fill any empty label indexes with "unknown"

@netlify
Copy link

netlify bot commented Jan 6, 2023

Deploy Preview for frigate-docs ready!

Name Link
🔨 Latest commit 33979a1
🔍 Latest deploy log https://app.netlify.com/sites/frigate-docs/deploys/63b7c488c88e3000089cc3ea
😎 Deploy Preview https://deploy-preview-4932--frigate-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@NateMeyer
Copy link
Contributor Author

The only reason OpenVino still has a unique labelmap is because the indexes are shifted one (index zero is "background"). Alternatively, we could decrement the detected label indexes in the OpenVino detector or make the index decrement a model parameter. That could get us to a common labelmap across all the detectors.

@blakeblackshear
Copy link
Owner

It's a small file and I don't expect to have all models share a common labelmap anyway.

@blakeblackshear blakeblackshear merged commit 8ac3114 into blakeblackshear:dev Jan 6, 2023
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.

None yet

2 participants