Skip to content
This repository was archived by the owner on Jan 7, 2025. It is now read-only.

Add classname display overlay - #84

Merged
corneliusboehm merged 5 commits into
masterfrom
feature/display-updates
Jan 22, 2021
Merged

Add classname display overlay#84
corneliusboehm merged 5 commits into
masterfrom
feature/display-updates

Conversation

@corneliusboehm

Copy link
Copy Markdown
Contributor

This PR copies over some general changes from #64 as long as that PR is still on hold.

  • Add font_scale and thickness arguments to the put_text method
  • Add DisplayClassnameOverlay operation that shows detected classes as a big text overlay (see screenshot)
  • Fix int parsing of camera_id in all scripts
  • Fix assignment of runtime_error in controller.py (the as statement wasn't working as expected)

facepalming

If you want to test the DisplayClassnameOverlay, add this to the display_ops in run_gesture_recognition.py:

        sense.display.DisplayClassnameOverlay(
            thresholds={
                "\"Sleeping\" gesture": 0.5,
                "Covering ears": 0.5,
                "Covering eyes": 0.5,
                "Facepalming": 0.5,
                "Nodding": 0.5,
                "Putting finger to mouth": 0.5,
                "Scratching": 0.5,
                "Shaking head": 0.5,
                "Waving": 0.5,
            }
        )

We could even add this to the script per default if people like it.

floriandotpy
floriandotpy previously approved these changes Jan 20, 2021

@floriandotpy floriandotpy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread sense/display.py Outdated
@corneliusboehm
corneliusboehm merged commit b26bed4 into master Jan 22, 2021
@corneliusboehm
corneliusboehm deleted the feature/display-updates branch January 22, 2021 16:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants