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

[New] Show device location using indoor positioning #462

Merged
merged 76 commits into from
Aug 20, 2024

Conversation

chriswebb09
Copy link
Contributor

@chriswebb09 chriswebb09 commented Jul 19, 2024

Description

This PR implements Show device location using indoor positioning in Map category.

  • swift/issues/5653 (internal test data in the issue)
  • common-samples/pull/3939

How To Test

Load map and wait for campus to load. You should see your location relative to the layout of the office.

Please note:

  • The sample data doesn't work in Building M due to no pathway table was surveyed, and not in Building L either maybe due to dead batteries. To see it in action, use the internal data in the issue linked above.
  • Both the indoorPositioningDefinition and the manual ips_positioning creation of the LDS should work, so make sure you test both condition branches

Screenshots

351891359-cf184145-a083-48f0-a5a0-3dbc6ca7c106

@chriswebb09 chriswebb09 self-assigned this Jul 19, 2024
@chriswebb09 chriswebb09 changed the title [New] - Show device location using indoor positioning [New] Show device location using indoor positioning Jul 19, 2024
@chriswebb09 chriswebb09 reopened this Jul 19, 2024
@chriswebb09 chriswebb09 marked this pull request as ready for review July 22, 2024 16:49
@chriswebb09 chriswebb09 requested a review from yo1995 July 22, 2024 16:49
@chriswebb09 chriswebb09 requested a review from philium July 23, 2024 17:53
@yo1995 yo1995 assigned yo1995 and unassigned chriswebb09 Aug 15, 2024
Copy link
Collaborator

@yo1995 yo1995 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some stuff I'm gonna address

@yo1995 yo1995 requested review from philium, a team and CalebRas and removed request for a team August 15, 2024 23:29
CalebRas
CalebRas previously approved these changes Aug 19, 2024
Copy link
Collaborator

@CalebRas CalebRas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@chriswebb09
Copy link
Contributor Author

😁

@yo1995
Copy link
Collaborator

yo1995 commented Aug 20, 2024

Thank you all for the feedback. Chris, great job for pulling all the pieces together!

@yo1995 yo1995 merged commit 1e5007b into v.next Aug 20, 2024
1 check passed
@yo1995 yo1995 deleted the chriswebb/New-ShowDeviceLocationUsingIndoorPositioning branch August 20, 2024 03:06
@chriswebb09
Copy link
Contributor Author

Thank you guys for getting it across the finish line!

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.

4 participants