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

add presetting of ublox position from LastGPSPosition #26083

Closed
wants to merge 3 commits into from

Conversation

gast04
Copy link
Contributor

@gast04 gast04 commented Oct 15, 2022

No description provided.

@robbederks
Copy link
Contributor

How bad is the TTFF if this last position is significantly wrong? Can imagine this happens quite often in the field

@gast04
Copy link
Contributor Author

gast04 commented Oct 16, 2022

I will check it, why you think that happens often? locationd updates LastGPSPosition every minute

@gast04 gast04 force-pushed the ublox-preset-location branch from 47251af to dbe9297 Compare October 16, 2022 22:06
@adeebshihadeh
Copy link
Contributor

adeebshihadeh commented Oct 16, 2022

I will check it, why you think that happens often? locationd updates LastGPSPosition every minute

Regardless of whether this happens often (or even at all), we should understand failure modes like this, especially for changes that appear to be a simple, strict improvement. This actually happens on every fresh boot since LastGPSPosition is a persistent param; you drive, LastGPSPositionis properly set, device shuts off after 30 hours offroad, you go to drive again, device boots up while you're driving, and now the position is off.

@gast04
Copy link
Contributor Author

gast04 commented Oct 17, 2022

yea, very true, didnt consider that case

@adeebshihadeh
Copy link
Contributor

So I think there's three things left for the u-blox improvement project:

  • understand the effects that varying degrees and combinations of wrong assistance data has on TTFF
  • optimize TTFF for the online case; <15s is good, <10s is great
  • test the offline TTFF case to ensure it's consistent

@gast04
Copy link
Contributor Author

gast04 commented Oct 19, 2022

I got some results, first of all this cannot be reliably tested inside the GPS test box, most likely cause of too strong signal as robbe mentioned. Different test scenarios inside the box give similar timing, which cannot be true. So tests have been done outside the box.

ttfm, time to first measurement (ubloxGnss)
ttfl, time to first location (gpsLocationExternal), basically ttff
values in seconds

scenario 1, no reset, flash backup available (very likely case)
ttfm: 6.29, ttfl: 4.28
ttfm: 8.40, ttfl: 5.03
ttfm: 7.80, ttfl: 3.92

scenario 2, no reset, flash backup available, online data (wrong location, 9miles off)  (very likely case)
ttfm: 7.93, ttfl: 3.22
ttfm: 8.68, ttfl: 2.84
ttfm: 13.58,ttfl: 9.33
ttfm: 9.48, ttfl: 2.58

scenario 2, device reset with online assistance data
ttfm: 26.70, ttfl: 28.85
ttfm: 38.11, ttfl: 39.04
ttfm: 23.61, ttfl: 25.25
ttfm: 25.01, ttfl: 25.85

scenario 3, device reset with online assistance data, with location preset
ttfm: 20.58, ttfl: 21.81
ttfm: 23.87, ttfl: 24.72
ttfm:  9.98, ttfl: 11.61
ttfm: 20.88, ttfl: 22.32

with reset and online data(wrong location, 181miles off)
ttfm: 93.30, ttfl: 97.25
ttfm: 19.90, ttfl: 19.86
ttfm: 139.31,ttfl: 142.05
ttfm: 81.95, ttfl: 82.01

flash backups are tolerant against location changes up to 20miles,
no noticeable time increase (tested inside box)

Summary, using online assist and flash backup, we can assure a ttff below 10seconds, will add this to this PR and remove the location change

@gast04
Copy link
Contributor Author

gast04 commented Oct 19, 2022

reverted the change, final testing outside the box showed that UTC time setting and the flash backup gives the best result without online assist data, we hit about 10-12seconds until first location, the timing issue was that the spoofed signal has a wrong time...

@robbederks
Copy link
Contributor

It might be nice to try to modify the testing box to be more alike to a real-life scenario, either by:

  • Attenuating the signal to the point that the natural noise in the ublox amp and ADC is strong enough
  • Adding gaussian (?) noise on top of the gps samples to mimic the real life signal quality that way

I guess the second method might be more desirable, since that would be most resistant to outside factors like ambient noise, temp,...

@gast04
Copy link
Contributor Author

gast04 commented Oct 20, 2022

there is an attenuation setting in the limeGPS, will try this one later today, about adding gaussian I need to check the limeCode,
but should be doable do, moved to testing outside the GPS box for now

@adeebshihadeh
Copy link
Contributor

@gast04 what's the conclusion on this? close for now?

@gast04
Copy link
Contributor Author

gast04 commented Oct 26, 2022

yes, I would close it for now, wanna bring the GPS box into the CI, and address the attenuation at a later point

@gast04 gast04 closed this Oct 26, 2022
@adeebshihadeh adeebshihadeh deleted the ublox-preset-location branch October 26, 2022 21:49
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.

4 participants