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

Corrupt GPS data after upgrading to "Improved GNSS module initialization" commit #186

Closed
gyrofx opened this issue Sep 17, 2023 · 9 comments

Comments

@gyrofx
Copy link

gyrofx commented Sep 17, 2023

After using the latest telelogger build (1505b85) GPS data coming from the device seems corrput.

image

The date field is always 000000, which causes parsing to fail and no GPS data to be sent or recorded.

Without the changes from this commit, everything works as expected.

image

Device: Freematics One+ Model B (pretty new), internal GPS, WIFI, built with platform.io on Windows

@canegru
Copy link

canegru commented Sep 19, 2023

Confirmed as well, I reverted the commit and it works as expected.

@dd310
Copy link

dd310 commented Sep 29, 2023

Even with the latest commit, the issue is not resolved (at least for me, ONE+ model A, external GNSS)
Can confirm that the latest commit of FreematicsPlus.cpp that works is still b39413a

@stanleyhuangyc
Copy link
Owner

stanleyhuangyc commented Oct 6, 2023

This commit seems have affect older batch of devices. Could you guys confirm the date of purchase of your device?

@stanleyhuangyc
Copy link
Owner

Avoided M10 GNSS commands from being sent as a workaround.

@dd310
Copy link

dd310 commented Oct 6, 2023

This commit seems have affect older batch of devices. Could you guys confirm the date of purchase of your device?

09/09/23

@canegru
Copy link

canegru commented Oct 6, 2023

This commit seems have affect older batch of devices. Could you guys confirm the date of purchase of your device?

09/12/23 here

@gyrofx
Copy link
Author

gyrofx commented Oct 6, 2023

This commit seems have affect older batch of devices. Could you guys confirm the date of purchase of your device?

07/28/23

@stanleyhuangyc
Copy link
Owner

stanleyhuangyc commented Oct 8, 2023

The issue is fixed now. It affects ONE+ devices with u-blox m10. It does not affect older batches with m8.

The cause:
Recently added m10 configuration commands disable GNSS' RMC sentence so no date value is reported but date is still validated in gpsGetData() in the library causing no GNSS data can be obtained in telelogger.

@dd310
Copy link

dd310 commented Oct 11, 2023

Just tried the latest commit and the issue seems indeed solved. Thanks!

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

No branches or pull requests

4 participants