-
Notifications
You must be signed in to change notification settings - Fork 107
[On hold] Add digit recognition example #64
base: master
Are you sure you want to change the base?
Conversation
# Conflicts: # realtimenet/display.py
# Conflicts: # sense/display.py # sense/downstream_tasks/digit_recognition/__init__.py
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?
|
Don't think I have seen this 😕 Do any of the other demos work? |
There was a problem hiding this 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
Co-authored-by: Florian Letsch <florianletsch@users.noreply.github.com>
Co-authored-by: Florian Letsch <florianletsch@users.noreply.github.com>
… into feature/digit-recognition
The final layer of both backbone networks was trained to recognize digits drawn in the air and some display logic is added.
General updates:
font_scale
andthickness
input_text
runtime_error
handling incontroller.py
, because it was not showing the relevant exception, but only thatruntime_error
was used before declarationHere'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.