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

DeviceOrientationCamera #4608

Closed
wants to merge 2 commits into from
Closed

Conversation

malaretv
Copy link
Contributor

@malaretv malaretv commented Nov 4, 2016

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:

  • iOS does not support screen orientation API. We could probably safely default to primary-landscape (90deg) as was done before.

- device orientation maps directly to camera coordinate space
- adjusts for screen orientation
@hpinkos
Copy link
Contributor

hpinkos commented Nov 4, 2016

Great, thanks for the pull request @malaretv! @bagnell can you take a look at this when you have a chance?

@pjcozzi
Copy link
Contributor

pjcozzi commented Jan 15, 2017

@emackey are you familiar with this code? Would you be able to review this?

@emackey
Copy link
Contributor

emackey commented Jan 16, 2017

@pjcozzi I'm not, unfortunately. It's on my list of things to investigate when I get back into VR stuff.

@pjcozzi
Copy link
Contributor

pjcozzi commented Jan 16, 2017

@bagnell could you review this then?

@pjcozzi pjcozzi requested a review from bagnell July 21, 2017 13:27
@hpinkos
Copy link
Contributor

hpinkos commented Oct 20, 2017

@bagnell can you review this when you get a chance?

@malaretv
Copy link
Contributor Author

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.

@hpinkos
Copy link
Contributor

hpinkos commented Nov 16, 2017

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!

@bagnell bagnell mentioned this pull request Nov 17, 2017
6 tasks
@pjcozzi
Copy link
Contributor

pjcozzi commented Nov 17, 2017

Also added to #3422. There's potential for VR work to pick up soon.

@cesium-concierge
Copy link

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 @cesium-concierge stop. If you want me to start again, just delete the comment.


I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.

🌍 🌎 🌏

1 similar comment
@cesium-concierge
Copy link

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 @cesium-concierge stop. If you want me to start again, just delete the comment.


I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.

🌍 🌎 🌏

@hpinkos
Copy link
Contributor

hpinkos commented Jan 11, 2019

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.

@hpinkos hpinkos closed this Jan 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants