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

u-Blox Neo-M9N GPS support #14509

Closed
eli-IF opened this issue Mar 27, 2020 · 11 comments
Closed

u-Blox Neo-M9N GPS support #14509

eli-IF opened this issue Mar 27, 2020 · 11 comments
Assignees
Labels

Comments

@eli-IF
Copy link

eli-IF commented Mar 27, 2020

Describe problem solved by the proposed feature
I ordered an mRo GPS that was listed as supported but it came with an u-Blox Neo-M9N chip and it is not plug and play. I used "gps status" in the mavlink console and it shows that gps status is not ok.

Describe your preferred solution
I would prefer to have a plug and play solution for this specific GPS. I assume this requires changing the driver for gps but I am not familiar enough to do this myself.

@dagar dagar added the bug label Mar 27, 2020
@dagar
Copy link
Member

dagar commented Mar 27, 2020

@bkueng any ideas here?

@dagar
Copy link
Member

dagar commented Mar 27, 2020

@pkocmoud FYI

@pkocmoud
Copy link
Contributor

Form Jordi:
So M9N doesn't support the UBX->SOL message anymore.
The new message that has to be enabled is the UBX->PVT

@LorenzMeier
Copy link
Member

@dagar Could you please assign Beat to issues for anything GPS related? We need to get better at assigning and follow up. Thanks!

@ryanjAA
Copy link
Contributor

ryanjAA commented Apr 15, 2020

Took a little trickery for some tests but pretty incredible unit after some indoor and outdoor testing. 29 satellites outside with not a great shot to the sky and 24 inside nowhere near any windows.

2FDAA3AB-84D4-41AA-80E3-4A7167902A33

@dagar dagar added this to the Release v1.11.0 milestone Apr 15, 2020
@dagar
Copy link
Member

dagar commented May 1, 2020

This is included in v1.11.

@dagar dagar closed this as completed May 1, 2020
@ryanjAA
Copy link
Contributor

ryanjAA commented May 1, 2020

I don’t think this should be closed yet. It doesn’t work yet.

If someone else wants to test an M9N and confirm that it works that would be great but as of yesterday we tested it and in the current state is still just gives a timeout without functionality. See PX4/PX4-GPSDrivers#57 (comment) for more details.

@dagar
Copy link
Member

dagar commented May 1, 2020

I don’t think this should be closed yet. It doesn’t work yet.

Thanks for the update. Let's try to keep conversation going outside of merged PRs so it isn't lost.

@dagar dagar reopened this May 1, 2020
@bkueng
Copy link
Member

bkueng commented May 4, 2020

@ryanjAA continuing here from PX4/PX4-GPSDrivers#57 (comment)

Whats the difference between this and master? If i flash master as of right now from QGC, it works. If i pull cdee27f in as i did from ~2 days ago, it does not. Normally is wouldn't matter but if this is trying to be put into 11 release this needs to work with that and not master.

Can you be more specific with the repos and commits you use? You mean Firmware master (which commit)? This will all go into the release, so if it works there we're all good. There was another issue that could cause issues with GPS, which you might have hit (#14771).

@ryanjAA
Copy link
Contributor

ryanjAA commented May 4, 2020

Well this is a little strange.

Master (at 8bc72fe, #14785) with GPS Debug settings changed that did not work as shown before (times out) is now working (see attached) but i let the unit sit (so the onboard battery maybe drained?). There isn't much else I can think of since the time out is the same (when switching bauds), it just now seems to at some point find in (near instantly) where before it was sitting for a minute or two and still not recognizing the unit. If you flash 8bc72fe, it works instantly and then I checked by manually by turning off the driver and then back on to see the Debug timeout messages.

Master (at cd59f95, #14788) that was pulled in from QGC does work

Master (at 70329ce, #14726) works with 4cc28b8, PX4/PX4-GPSDrivers#55 (GPS Driver pulled in and hz manually changed to 5hz to test max sat usage [meas 100 to 200]).

Beta 746b312, #14609 (from qgc) does not work.

Master as of writing (466b5db, #14784) pulled from QGC works (and frequency is at 5hz).

If you're happy with that, I'd say it works and close. If you want me to test against #14771, let me know.

Screen Shot 2020-05-04 at 10 21 09 AM

@bkueng
Copy link
Member

bkueng commented May 5, 2020

Ok, thanks for the testing. I say we leave it at that since master works, and if you notice issues (such as reliability in module detection), open a new issue.

@bkueng bkueng closed this as completed May 5, 2020
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

6 participants