Skip to content

Hand detect data stream #813

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Hand detect data stream #813

wants to merge 3 commits into from

Conversation

DanielMatyas
Copy link

Detect number of fingers extended on data stream.

Removed Snap! button and Process tab from the notebook. Changed image
display from label to a separate window, in order to have better frame
rate.

The camera requests only the depth image.

After clicking on Start Video, the depth image is displayed
continuously. Every 1 second a process of finger detection is started on
a different core. After finishing the calculation the bounding box of
the hand and the number of detected fingers is updated.

Signed-off-by: Daniel Matyas <daniel.matyas@analog.com>
First of all, it is faster, because there is no GUI, only computation.

Used depth image directly for computation; did not convert image to RGB.

Made max distance of the bit image dynamic. It depends on the closest
pixel.

Changed hand center to the point farthest from the background

Made cdist_threshold dynamic. It is inversely proportional to
the max distance of bit image.

Selected the vertex which is farthest from the hand center on every
finger.

Neglected the vertices which are below the wrist.

Signed-off-by: Daniel Matyas <daniel.matyas@analog.com>
@DanielMatyas DanielMatyas force-pushed the hand-detect-data-stream branch from 36bae68 to 6b7ccdf Compare September 28, 2023 11:17
@dNechita
Copy link
Collaborator

Hi @DanielMatyas! Just checking if this is this ready to be reviewed. Is it?

@DanielMatyas DanielMatyas force-pushed the hand-detect-data-stream branch from 6b7ccdf to 8b2b83e Compare October 25, 2023 11:20
@DanielMatyas
Copy link
Author

DanielMatyas commented Oct 25, 2023

Yes. It is ready for review @dNechita .

Added more commentaries and made code look somewhat nicer.

Signed-off-by: Daniel Matyas <daniel.matyas@analog.com>
@DanielMatyas DanielMatyas force-pushed the hand-detect-data-stream branch from 8b2b83e to a008da5 Compare November 13, 2023 08:49
@SRaus SRaus changed the base branch from master to main January 9, 2024 15:14
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.

4 participants