-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
DeviceOrientationCamera #4608
DeviceOrientationCamera #4608
Conversation
- device orientation maps directly to camera coordinate space - adjusts for screen orientation
@emackey are you familiar with this code? Would you be able to review this? |
@pjcozzi I'm not, unfortunately. It's on my list of things to investigate when I get back into VR stuff. |
@bagnell could you review this then? |
@bagnell can you review this when you get a chance? |
Any word on this? I'll have some downtime soon and wiling to revisit and work with y'all to get a fix in. If the pull request is unacceptable as-is I'm open to feedback as to approaches that could work better! There's also https://github.com/dorukeker/gyronorm.js if willing to offload getting consistent gyroscope values across devices to a third-party lib. |
Sorry it's taken so long for us to get to this @malaretv. We'll try to carve out some time to review it soon. Thanks! |
Also added to #3422. There's potential for VR work to pick up soon. |
Thanks again for your contribution @malaretv! No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy? I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome. 🌍 🌎 🌏 |
1 similar comment
Thanks again for your contribution @malaretv! No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy? I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome. 🌍 🌎 🌏 |
I'm going to close this PR because no commits have been made in over 2 years and I'm sure it needs a lot of updating before we can review it again. Thanks for putting the time into this @malaretv! We're hoping to make some WebXR improvements this year and will certainly use the changes in this PR as a reference. |
WIP
An attempt to address the device orientation issues laid out in #4135
Instead of repeatedly applying deltas of the device orientation to current camera orientation this approach maps it directly to the scene (taking into account device orientation).
The overall behavior is improved as far as responding to orientation changes. Now need to look into better orientation on intializing -- shouldn't have to know where true north is to find the globe :)
Other potential issues: