Skip to content

PB-521: Enabled geolocation high accuracy#882

Merged
ltshb merged 1 commit intodevelopfrom
bug-PB-521-goelocation-high-precision
Jun 6, 2024
Merged

PB-521: Enabled geolocation high accuracy#882
ltshb merged 1 commit intodevelopfrom
bug-PB-521-goelocation-high-precision

Conversation

@ltshb
Copy link
Contributor

@ltshb ltshb commented Jun 6, 2024

While testing geolocation it often results to really poor accuracy, sometimes
up to several kilometers !

Test link

While testing geolocation it often results to really poor accuracy, sometimes
up to several kilometers !
@github-actions github-actions bot added the bug label Jun 6, 2024
@cypress
Copy link

cypress bot commented Jun 6, 2024

Passing run #2388 ↗︎

0 207 20 0 Flakiness 0

Details:

PB-521: Enabled geolocation high accuracy
Project: web-mapviewer Commit: 1d3fa1354a
Status: Passed Duration: 05:46 💡
Started: Jun 6, 2024 9:06 AM Ended: Jun 6, 2024 9:11 AM

Review all test suite changes for PR #882 ↗︎

Copy link
Contributor

@pakb pakb left a comment

Choose a reason for hiding this comment

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

Looking forward using it next time I go outdoor 😄

@ltshb
Copy link
Contributor Author

ltshb commented Jun 6, 2024

I tested it today while mountain biking and here is my test impressions:

  • After enabling, I got within a few seconds an initial position about ~1km off
  • In less than 10-15 seconds I got a higher precision within ~50m
  • In less than 30s I got a high precision less ~10m
  • I was on a small trail in the forest while testing with clear wheater
  • later on I used watch position while biking at 40km/h down a road outside the forest and the geolocation work very well and was precise (although a bit slower that I was biking)

@ltshb ltshb merged commit 0cbcb97 into develop Jun 6, 2024
@ltshb ltshb deleted the bug-PB-521-goelocation-high-precision branch June 6, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants