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

Hand detect data stream #813

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
Collaborator

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 <[email protected]>
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 <[email protected]>
@dNechita
Copy link
Collaborator

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

@DanielMatyas
Copy link
Collaborator 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 <[email protected]>
@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