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

Request: BodyPoseBoneTransforms can parse a BodyPose Asset in either world or local space #89

Open
cloudcover opened this issue Oct 1, 2024 · 1 comment

Comments

@cloudcover
Copy link

cloudcover commented Oct 1, 2024

I am currently dealing with an issue where I am using the retargeted humanoid from the Body Tracking for Fitness examples to load in a BodyPose that I have pre-recorded. The issue is that I cannot place the pose guide in the location that I would like to, because the serialized data for a BodyPoseData Asset is ingested by the BodyPoseBoneTransforms Component in world space.

I personally don't feel that this is very useful in general. It seems to me that if one ever wanted to load in pre-recorded BodyPoseData Assets in the spirit of why this toolset was created (for matching poses in fitness games), they would always be expected to be parsed in local space, otherwise every pose would be in a different location relative to where they incidentally happened to be captured in the capturer's tracking space on startup at runtime.

Nevertheless, it seems prudent that BodyPoseBoneTransform should either take in a local space BodyPose by default, or allow you the top-level option to stipulate upon getting a valid bodyPose reference. I got where I needed to go by faking an input to BodyPoseBoneTransforms that subtracts the root offset of the first bone from every bone in the list as a faked world space position, but this is a hack and definitely not ideal.

@cloudcover cloudcover changed the title Request: BodyPoseBoneTransforms can export BodyPoseData Asset in either world or local space Request: BodyPoseBoneTransforms can parse a BodyPose Asset in either world or local space Oct 1, 2024
@sohailshafiiWk
Copy link
Contributor

Thank you for the suggestion. We also think this would be useful for other developers and have added it to our feature backlog.

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

No branches or pull requests

2 participants