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

App doesn't work with ublox LEA M8P #7

Closed
kalon33 opened this issue Aug 24, 2017 · 34 comments
Closed

App doesn't work with ublox LEA M8P #7

kalon33 opened this issue Aug 24, 2017 · 34 comments
Assignees
Labels

Comments

@kalon33
Copy link

kalon33 commented Aug 24, 2017

Using a USB GPS module based on a ublox LEA M8P, I got the error "too many connection problems". GPS is working properly with u-center on Windows 10.

@kalon33
Copy link
Author

kalon33 commented Aug 24, 2017

screenshot_2017-08-24-22-57-14
screenshot_2017-08-24-22-56-33

@freshollie
Copy link
Owner

I see this device supports NMEA. Is it possible I could get a log? I have only been testing this on http://usglobalsat.com/p-688-bu-353-s4.aspx.

Do any other apps in the playstore work with this device?

@freshollie freshollie self-assigned this Aug 25, 2017
@freshollie freshollie added the bug label Aug 25, 2017
@kalon33
Copy link
Author

kalon33 commented Aug 25, 2017 via email

@freshollie
Copy link
Owner

For now I'm going to concede that I will not be able to make this device work due to it having a vast array of protocols which need to be selected and I have no way of testing.

If you want this app to work I suggest buying the GPS using I linked.

Other options are you edit the driver to work with your device and then send me a pull request once completed.

@kalon33
Copy link
Author

kalon33 commented Aug 25, 2017 via email

@freshollie
Copy link
Owner

I mean yeah I could probably edit the driver if I had the device, but editing a driver for a device which I don't own would be nearly impossible.

Can you send me the android log file after you get the too many connection problems?

That error is raised when the device is not outputting any info at all. All this service does is literally read the data output directly strings, and it sounds like your device is sending none (Possibly waiting for the correct start signal)

I don't want you to send me anything expensive as I have no guarantee of this working. Though if you are still serious about sending me a device to work with then that can be arranged.

@kalon33
Copy link
Author

kalon33 commented Aug 25, 2017 via email

@freshollie
Copy link
Owner

freshollie commented Aug 25, 2017

Android has its own USB wrapper which doesn't require access to /dev/tty***. As this cannot be done without root. See https://developer.android.com/reference/android/hardware/usb/package-summary.html

The logcat would be perfect. I have several debug lines which I can read from the output and maybe understand the problem.

Can you cat /dev/tty*** on your computer with the devices plugged in and also send me the output?

@freshollie freshollie changed the title UsbGPS stopped: too many connection problems App doesn't work with ublox LEA M8P Aug 25, 2017
@freshollie freshollie reopened this Aug 25, 2017
@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

So here is the cat /dev/ttyACM0

adb logcat to come.

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

adb logcat -d

Is that the one you want?

@freshollie
Copy link
Owner

Perfect. The cat shows that the device is outputting the correct data, the logcat shows where the error is. I will look into fixing an release a test build today!

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

Side question, does the app parses all NMEA messages? Does it support all GNSS constellation messages (from GPS, but also from GLONASS, Beidu and Galileo systems)? This device supports them all (as stated in the pdf file I sent).

Thanks for the fix :)

@freshollie
Copy link
Owner

freshollie commented Aug 27, 2017

I will get back to you on that. Please download and install this apk and do another adb logcat. Notice the error message on screen has now changed.

@freshollie
Copy link
Owner

As per. NmeaParser the only information which is actually collected from the device is

  • GPGGA
  • GPRMC

These provide all the needed information to update the android GPS location.

All other information can be written to a file if specified (Enable/Disable NMEA log)

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

Indeed, message has changed:
screenshot_2017-08-27-14-57-33

Here is the new logcat

Thanks

@freshollie
Copy link
Owner

Thanks, can you download the app from the same link and send me the logcat again :)

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

Here it is :)

@freshollie
Copy link
Owner

Same again please, what I am trying to do is work out what interface the device requires.

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

Here it is again :)

@freshollie
Copy link
Owner

Hey that was working! Leave it going!

@freshollie
Copy link
Owner

The log shows that you disconnected the device.

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

Android disconnected itself, asking me again if I authorize usbgps to use it. Meanwhile (a few seconds after the message, before I choosed OK, I got an error). I launched it again after the logcat, left it running. I have NMEA data in the log, but N/A for everything: number of satellites, accuracy, location, altitude and Gps Time.

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

Other android apps that are normally giving satellites list with internal GPS doesn't provide any info now.

@freshollie
Copy link
Owner

Reading the log, it looks like the NMEA protocol being read back has not been implemented into the driver.

Other android apps that are normally giving satellites list with internal GPS doesn't provide any info now.

I don't understand what you mean this.

@freshollie
Copy link
Owner

freshollie commented Aug 27, 2017

I've built an APK at the same link to work for added NMEA set, try that. Leave the GPS somewhere where it can get a fix. The log showed no GPS fix.

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

In some other android GPS apps (for internal GPS devices), you can get a list of current satellites used, and various info about GPS. For now I don't have them with the external one. Is that expected?

Now I have proper position fix, with altitude and number of satellites used in UsbGPS (the latter one not outside UsbGPS)

@freshollie
Copy link
Owner

Yes, that is expected due to the driver not interpreting the list of satellites from the device, only the number.

So the device is now working perfectly?! :))

@freshollie
Copy link
Owner

If you want an interpretation of the satellite positions, please make a separate issue. It will take much longer for me to produce, though it is a long term goal.

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

yep, that's now working :)

Is that expected that GPS time is a long integer, not usual time format? I'll create this issue, and, if you mind, another one to have some basic options like the ones available for sirf devices :)

@freshollie
Copy link
Owner

"Gps Time" should be the same length as "Sys Time" as can be seen from the screenshots. That's great this is now working, I am surprised.

freshollie added a commit that referenced this issue Aug 27, 2017
@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

so this is OK, both strings look like the same kind of thing. What's the difference between altitude (MSL) and altitude? I get both from my internal chip and only MSL one from external device (but not labeled as such).

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

I don't have fix type info neither. Is that expected?

@kalon33
Copy link
Author

kalon33 commented Aug 27, 2017

neither VDOP/HDOP/PDOP...

@freshollie
Copy link
Owner

Do the altitude numbers differ? I just read what comes from the nmea and add it to a location object.

https://developer.android.com/reference/android/location/Location.html

Notice this object does not have the ability to set satellites or any of that info. I am pretty sure this is a limitation of android.

freshollie added a commit that referenced this issue Dec 4, 2017
freshollie added a commit that referenced this issue Dec 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants