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.

Sign up for GitHub

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

Jump to bottom

Don't use neck length if head tracker doesn't have position #1019

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Don't use neck length if head tracker doesn't have position #1019

wants to merge 1 commit into from

Conversation

Erimelowo
Copy link
Member

#1008

This also rectifies the check to not care about rotation. It cared about rotation because it was initially only supposed to apply for OSCTrackers, but we need to do the same for IMU head trackers for VTubing/mocap.

@ImUrX ImUrX linked an issue May 2, 2024 that may be closed by this pull request
Don't use neck length when head tracker doesn't have position #1008
Open
@ImUrX ImUrX added Type: Enhancement Adds or improves a feature Area: Server Related to the server Area: Skeletal Model Deals with the model of the skeleton and its pose labels May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ImUrX ImUrX ImUrX approved these changes

@ButterscotchV ButterscotchV Awaiting requested review from ButterscotchV ButterscotchV is a code owner

@Eirenliel Eirenliel Awaiting requested review from Eirenliel Eirenliel is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
Area: Server Related to the server Area: Skeletal Model Deals with the model of the skeleton and its pose Type: Enhancement Adds or improves a feature
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Don't use neck length when head tracker doesn't have position
2 participants
@Erimelowo @ImUrX