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

Phone camera is not rendering as background when the glTF scene contains a camera #51

Open
nlsdvl opened this issue Aug 9, 2024 · 0 comments · Fixed by #58
Open

Phone camera is not rendering as background when the glTF scene contains a camera #51

nlsdvl opened this issue Aug 9, 2024 · 0 comments · Fixed by #58
Labels
bug Something isn't working
Milestone

Comments

@nlsdvl
Copy link
Contributor

nlsdvl commented Aug 9, 2024

When loading the sofa scene, when the floor is detected, the scene is instantiated as expected.

However if the scene contains one or more camera, it gets instantiated and effectively breaks AR:

  • the phone's camera device is no longer used as background
  • the viewport becomes static, and pose tracking no longer works

@Etiennefaivredarcier what is the expected behavior for such a case ? I assume the camera node in the gltf document in this context is not forbidden by the spec, and the behavior in the specification is undefined, so it can be ignored.

It seems this was addressed for plane tracking in TrackableGeometric.cs, but the floor tracking implementation doesn't handle this case.

The issue exists potentially on all trackable types used to anchor a scene containing a camera.
This could also be extracted/refactored to avoid redundancy.

@nlsdvl nlsdvl added the bug Something isn't working label Aug 9, 2024
@nlsdvl nlsdvl changed the title Phone camera is not rendering as background when the gilt scene contains a camera Phone camera is not rendering as background when the glTF scene contains a camera Aug 9, 2024
@nlsdvl nlsdvl added this to the Version 1.0.0 milestone Nov 5, 2024
@nlsdvl nlsdvl linked a pull request Nov 5, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging a pull request may close this issue.

1 participant