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

remove "Mouth21" #159

Merged
merged 1 commit into from
Feb 11, 2025
Merged

remove "Mouth21" #159

merged 1 commit into from
Feb 11, 2025

Conversation

hunminkim98
Copy link
Contributor

@hunminkim98 hunminkim98 commented Feb 11, 2025

Hello Dr. David,

While testing DeepSort with COCO_133, I noticed a poor connection between the Lwrist and Mouth nodes. Upon further investigation, I realized that the face should have 68 keypoints, but the current COCO_133 in the skeleton.py contains 69. I suspect this discrepancy may have caused the issue.

To test this hypothesis, I removed the last mouth node (Mouth21). As a result, the tracking now appears to be working more reasonably!

Current version: 69 face keypoints (Jaw1 (id=13) to Mouth21 (id=91))
image

Modified version: 68 face keypoints (Jaw1 (id=13) to Mouth20 (id=90))
image

Thanks for reading!

Best regards,
HunMin Kim

@davidpagnon davidpagnon merged commit 5861b43 into perfanalytics:main Feb 11, 2025
8 checks passed
@davidpagnon
Copy link
Collaborator

Thanks for spotting this! Merged to main.

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