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

add OneHand10K dataset #52

Merged
merged 7 commits into from
Aug 13, 2020
Merged

add OneHand10K dataset #52

merged 7 commits into from
Aug 13, 2020

Conversation

liuxin9608
Copy link
Collaborator

Add OneHand10K dataset

@liuxin9608 liuxin9608 requested a review from jin-s13 August 4, 2020 12:06
@CLAassistant
Copy link

CLAassistant commented Aug 4, 2020

CLA assistant check
All committers have signed the CLA.


self.ann_info['flip_pairs'] = []

self.ann_info['use_different_joints_weight'] = False
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not specific to this pr, but this name reads strange. maybe "use_different_joint_weights" is better? btw what does this parameter mean @jin-s13

image

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aha. Yes. use_different_joint_weights seems better.
This will assign different loss weight to different joints (for example, pay more attention to the accuracy of wrists.).

@jin-s13
Copy link
Collaborator

jin-s13 commented Aug 6, 2020

Please do not merge this branch, until the model checkpoint (.pth) is provided.

@jin-s13
Copy link
Collaborator

jin-s13 commented Aug 13, 2020

We have validated the model accuracy and provided the model, see README.md.

@innerlee innerlee merged commit 7f36f9c into master Aug 13, 2020
@innerlee innerlee deleted the hand branch August 13, 2020 11:33
rollingman1 pushed a commit to rollingman1/mmpose that referenced this pull request Nov 5, 2021
shuheilocale pushed a commit to shuheilocale/mmpose that referenced this pull request May 6, 2023
Co-authored-by: jinsheng <jinsheng@sensetime.com>
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