Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

[On hold] Add digit recognition example #64

Draft
wants to merge 19 commits into
base: master
Choose a base branch
from

Conversation

corneliusboehm
Copy link
Contributor

@corneliusboehm corneliusboehm commented Dec 17, 2020

The final layer of both backbone networks was trained to recognize digits drawn in the air and some display logic is added.

General updates:

  • Support font_scale and thickness in put_text
  • Fix the runtime_error handling in controller.py, because it was not showing the relevant exception, but only that runtime_error was used before declaration

Here's a perfect run where all digits are recognized (even though it doesn't always work): https://drive.google.com/file/d/1pEM0a2OYGxPvuqC9gvft8RQKb9iVdtkY/view?usp=sharing

Please try this out yourself and let me know if it's good enough to be a visible example in the README.

@floriandotpy
Copy link
Contributor

I'm pretty sure this is unrelated to your example, but in a fresh installation of the repo and environment, I can't get the demo to work. It prints two lines, does not open a window, does not show an error, and simply terminates.

Have you seen this before?

$ PYTHONPATH=./ python scripts/run_digit_recognition.py --camera_id=1
Starting inference
Stopping inference

$ 

@corneliusboehm
Copy link
Contributor Author

Don't think I have seen this 😕 Do any of the other demos work?

Copy link
Contributor

@floriandotpy floriandotpy left a comment

Choose a reason for hiding this comment

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

Code looking good. I left minor comments.

I've tested it now, but unfortunately, it didn't work really well for me. I feel like our old airdigits demo was way more reliable and I also didn't have to be so careful with the timing of my motion.
https://drive.google.com/file/d/1kZvOM1iCSyM3fq3alcK8jcNK5Ya8xbOP/view?usp=sharing

sense/display.py Outdated Show resolved Hide resolved
sense/display.py Outdated Show resolved Hide resolved
Co-authored-by: Florian Letsch <florianletsch@users.noreply.github.com>
corneliusboehm and others added 2 commits January 18, 2021 14:48
Co-authored-by: Florian Letsch <florianletsch@users.noreply.github.com>
@corneliusboehm corneliusboehm marked this pull request as draft February 19, 2021 11:26
@corneliusboehm corneliusboehm changed the title Add digit recognition example [On hold] Add digit recognition example Feb 19, 2021
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.

None yet

3 participants