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

local pose used for EstimateFeatureMapQualityForHosting #81

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pixlhero
Copy link

@pixlhero pixlhero commented Aug 4, 2022

Updated documentation and sample usage for EstimateFeatureMapQualityForHosting.

The issue is that the documentation doesnt specify and the sample uses the global pose of the camera. But actually the local pose is expected. This hasn't been an issue in the sample, because the AR Session Origin is at (0,0,0). This is very common, but there are use-cases where the Session Origin has to be moved, which would break the sample when using EstimateFeatureMapQualityForHosting.

Updated documentation and sample to use the local camera pose instead of the global camera pose.
@pixlhero pixlhero marked this pull request as ready for review August 4, 2022 08:11
@devbridie devbridie self-assigned this Aug 4, 2022
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.

2 participants