-
Notifications
You must be signed in to change notification settings - Fork 53
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
An error of poses_diff() function #6
Comments
yes, I also find this error, after fixing it , the accuracy gets better. |
Thanks very much for pointing out this issue. I had corrected them in the current version. As both of you had mentioned, such a correction did an improvement in SHREC data (for coarse data: It was improved from 94.6 to 94.8 with 1.82M parameters and from 91.8 to 94.1 with 0.15M parameters). |
great job! very fast and high accuracy. And have you ever tried this model on other dataset ,especially the NTU RGB+D dataset. which is very hot for skeleton based action recognition. and how is the performance |
Sorry, I may not try it since the skeleton contains much noise and loses context information, it is not as good as only using RGB in body action recognition and I stop further investigate it. This work was originally designed for hand gesture as we could obtain a more accurate 3D hand pose and it may not be related to context information. If you are working on NTU, stacking more modules (e.g., 2s-AGCN) might help to boost the performance of NTU up to ~96+(CV) and ~90+(CS). |
@pengfeiZhao1993 @muxizju Can you please explain how to run the SHREC dataset. |
Got it! Thanks |
the subtract operation subtract x[:, :1, ...] and x[:, :-1, ...], while the first input might be wrong, it should be x[:, 1:, ...]. Current operation in fact subtract the first postion for all frames with array broadcast.
I have test that train the lite network on SHREC coarse data, the acc of validation is almost the same while the acc of training dataset is lower than validation. Which might be the reason of too much dropout. There might be still some space for improving.
If I'm wrong, please let me know
The text was updated successfully, but these errors were encountered: