mmhuman3d icon indicating copy to clipboard operation
mmhuman3d copied to clipboard

Question about hybrik code

Open dldaisy opened this issue 2 years ago • 2 comments

Dear authors, Thanks for the great work. I have a question about the below code. It seems that you have adapted the original code to implement naive hybrik. You change the final_pose_skeleton to relative postition in if train branch as indicated below: https://github.com/open-mmlab/mmhuman3d/blob/2ccdd2484353a77efe27508d207f6ae5349506ef/mmhuman3d/models/utils/inverse_kinematics.py#L71 However, in the code snippet below, you seemed subtract the rotated parent position from the relative final skeleton(instead of absolute position) to get a relative joint position, which doesn't make sense to me: https://github.com/open-mmlab/mmhuman3d/blob/2ccdd2484353a77efe27508d207f6ae5349506ef/mmhuman3d/models/utils/inverse_kinematics.py#L135 Can you explain why you can use relative final_pose_skeleton here? Thanks a lot!

dldaisy avatar Jul 06 '22 10:07 dldaisy

Hi @dldaisy, thanks for you point out this. This line looks like a redundant code because the element of the children map will not be -3. I have noticed that the official codes have made some changes, and I will check it. For naive HybrIK, the authors have released it in this issue.

ttxskk avatar Jul 20 '22 06:07 ttxskk

That's True. Thanks for your kind reply!

dldaisy avatar Jul 21 '22 01:07 dldaisy