search

orangeduck / BVHView

A simple viewer for the .bvh animation file format written using raylib.
MIT License
254 stars 14 forks source link

Error parsing a BVH file #3

Closed enekoatxa closed 4 months ago

enekoatxa commented 4 months ago

Hello, this is my first issue I have ever open, so forgive me if there's any error in form, I'll try to be the clearest I can.

I have tried the web build of BVHView, and I have been successful when importing a BVH file I had around, an instance from a public dataset. However, when I try to open another BVH that I recorded on my own using different software, the program returns a parsing error. errorBVH

I don't understand why does this error happen, because line 9:13 is not a place where a newline should be. The same BVH file can be opened in Blender, and it does not return errors. I attached the BVH file if it can be useful.

Thanks in advance. 000_actions.zip

orangeduck commented 4 months ago

Hi @enekoatxa,

Thanks for opening an issue! The problem is due to the . in the joint name i.e. spine.001. I've pushed up a fix to the repo that allows for . in names and so parses this file successfully: https://github.com/orangeduck/BVHView/commit/5cb5eb4d8b17d68752823daf99407c72145409b8

If you pull down the latest version and compile it yourself it should open your file.

I will update the web version and pre-compiled windows binaries next week (I'm traveling at the moment) - in the meantime if you re-name the joints so they don't include . in the name the file should load in the current web version.

Hope that helps,

Dan

enekoatxa commented 4 months ago

Thank you so much Dan! Both for the quick response and the quick fix!