Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

mocap head uses nose #9005

Merged
merged 1 commit into from
Oct 8, 2023
Merged

mocap head uses nose #9005

merged 1 commit into from
Oct 8, 2023

Conversation

HexaField
Copy link
Member

@HexaField HexaField commented Oct 6, 2023

Summary

🤖 Generated by Copilot at c956751

Improved head rotation calculation for motion capture pose. Used nose landmark instead of eyeCenter in solveHead function in solveMotionCapturePose.ts.

References

closes #insert number here

Explanation

🤖 Generated by Copilot at c956751

  • Simplify the input and computation of the head rotation from the mocap landmarks in solveMotionCapturePose (link, link, link, link)

🤖 Generated by Copilot at c956751

We'll sail the seas with motion capture
And solveHead with the nose
We'll simplify the function's input and output
And heave away on the count of three

QA Steps

List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.

Checklist

  • If this PR is still a WIP, convert to a draft
  • When this PR is ready, mark it as "Ready for review"
  • ensure all checks pass
  • Changes have been manually QA'd
  • Changes reviewed by at least 2 approved reviewers

@HexaField HexaField merged commit 674fa44 into dev Oct 8, 2023
12 of 14 checks passed
@HexaField HexaField deleted the mocap-head-uses-nose branch October 8, 2023 03:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant