facebookresearch / AnimatedDrawings

Code to accompany "A Method for Animating Children's Drawings of the Human Figure"
MIT License
10.56k stars 926 forks source link

Customize the BVH redirection file, the result gif display is very wrong #170

Closed iconnoci closed 1 year ago

iconnoci commented 1 year ago

I exported BVH from the fbx file to use, and re-wrote two configuration files, but the result is very weird, I read https://github.com/facebookresearch/AnimatedDrawings/blob/main/examples/config you wrote /README.md#motion-config-file, I feel that the configuration file is not wrong, but the result is wrong, I would like to ask you to see where there is a problem

walk.zip

hjessmith commented 1 year ago

What happens if you change the projection methods from the upper limbs and lower limbs from frontal to sagittal? Can you also please share the character-specific files (joint overlay and char_config.yaml), as well as a screen capture of the motion being perform by the original skeleton?

iconnoci commented 1 year ago

bounding_box.zip I have changed the projection method frontal, sagittal and pca, and all the files in garlic_out are compressed for you. 屏幕截图 2023-05-23 071403

iconnoci commented 1 year ago

Thank you for your reply

hjessmith commented 1 year ago

Okay, I think the problem may lie within the BVH parsing code. Is it easy for you to export another BVH from the .fbx, one in which each joint (excluding the root) has only rotation channels and not the position channels? If so, that would be helpful before I start debugging

iconnoci commented 1 year ago

Sorry, I use blender to import fbx and then export bvh, blender exports bvh, there is no option to export only rotation information but not position information.

iconnoci commented 1 year ago

walk2.zip Do you see if this fits your requirements? I deleted the position information of the animation frame by frame, and kept the position information of the hips.

hjessmith commented 1 year ago

That must have been a lot of work.

The problem was as I suspected. I've created a branch with a fix for you to try: https://github.com/facebookresearch/AnimatedDrawings/tree/bvh_fix

Please try running the code on that version and let me know if the results are as you expect. I see this video

iconnoci commented 1 year ago

good job it works video