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

Skip non-directory files in dataset creation (like .gitignore) #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marzzelo
Copy link

I propose some minor modifications (which do not affect compatibility with the video tutorial) and avoid a bug during serialization.

  • Add max_num_hands parameter to Hands class (set to 1), avoiding erroneous detection of a second hand (this sometimes occurs when the face appears in the frame during sampling).
  • (Bug Fix) Skip non-directory files that may be present under ./data folder during dataset creation (like .gitignore)
  • Display some status messages during serialization (the system seems frozen when many classes are processed).
  • Added .gitignore to exclude the generated pickle file and .venv & IDE cache directories.

…tory files in dataset creation (like .gitignore). Display some status messages during serialization.
@GLWmax
Copy link

GLWmax commented Feb 17, 2024

I would like to use it two-handed, but in your code I kept getting crashes. Could you help me use an extra code for two hands?

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.

2 participants