search

ros-drivers / nmea_navsat_driver

ROS package containing drivers for NMEA devices that can output satellite navigation data (e.g. GPS or GLONASS).
BSD 3-Clause "New" or "Revised" License
228 stars 270 forks source link

GNSS strings #15

Closed emilihb closed 6 years ago

emilihb commented 9 years ago

The current parser assumes $GPxxx strings only, which doesn't work with ublox 8. May I suggest to change the regular expression to '^\$G[N|P].\[0-9A-Fa-f]{2}$' to fix this?

ablakey commented 9 years ago

Do you have example strings emitted by the ublox 8? I'm curious how they're departing from the NMEA format. If I can read your regex properly, they're often emitting $GNxxx ?

valschmidt commented 8 years ago

It is also not uncommon for GPS aided INSs to produce an IN prefix (e.g. $INGGA). So the expression could be modified to '^\$[I|G][N|P].[0-9A-Fa-f]{2}$' (untested).

johschmitz commented 8 years ago

GP is for GPS only GN is for multi constelation GNSS systems (GPS,GLONASS,GALILEO) See the NMEA specification.

Anybody has a regex which can also parse ublox binary messages?

blmhemu commented 7 years ago

Go to /opt/ros/indigo/lib/python2.7/dist-packages/libnmea_navsat_driver (may be different for yours just type 'locate parser.py' ) and replace the parser.py in that folder with the one present in this git "https://github.com/ros-drivers/nmea_navsat_driver/blob/jade-devel/src/libnmea_navsat_driver/parser.py").

jack-oquin commented 7 years ago

Note that modifying files in the /opt/ros/... directory tree is only a temporary expedient. Future package updates can overwrite them.

evenator commented 6 years ago

This appears to have been fixed in 904ea247aa0c5305facf71811defc846394b28d3