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

Add GAN V4 gyroscope support #462

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

shreyasminocha
Copy link

@shreyasminocha shreyasminocha commented Jan 19, 2025

  • Make the virtual cube move with the smart cube
    • Make it move correctly
  • Fix twisty scene camera location
  • Allow toggling gyroscope from settings
  • Refactor code for production

Stretch:

  • Smooth animation across orientation updates
  • Use axis speeds when rotating

Out of scope:

  • Use gyroscope to inform virtual reconstruction (VR) orientation
  • Include gyroscope data in VR

Related: #442 and #213. Hopefully, once this PR is ready, it will be easier to add support for other smart cubes.

@shreyasminocha shreyasminocha marked this pull request as draft January 19, 2025 01:52
@shreyasminocha
Copy link
Author

Currently, the quaternion I'm constructing from the BLE data is incorrect: it doesn't mirror the orientation of the smart cube. Paging @afedotov (author of #419) in case they know what's wrong.

@shreyasminocha
Copy link
Author

Okay, I managed to fix the quaternion parsing issue thanks to @afedotov's gan-web-bluetooth.

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

Successfully merging this pull request may close these issues.

1 participant