Ask Your Question
2

Is there a way to use the nmea_gps_driver on electric version of ros?

asked 2012-03-29 13:41:54 -0500

BetoCC011 gravatar image BetoCC011
41 4 4 7

updated 2012-03-29 16:17:34 -0500

Eric Perko gravatar image Eric Perko flag of United States
6842 39 76 135
http://ericperko.com/

In the documentation, they say i can only use this driver in fuerte version or newer, due to the dependency on sensor_msgs/TimeReference

Is it possible to chance the driver so it doenst have this depency, or to chance ros electric to have this depency?

Thanks.

edit retag flag offensive close delete

2 Answers

Sort by ยป oldest newest most voted
2

answered 2012-03-29 15:40:32 -0500

joq gravatar image joq
15416 89 169 295
http://www.austinrobot.co...

updated 2012-04-06 11:29:46 -0500

From a glance at the code, it certainly looks possible.

If you have time to work on it, check out the git sources and make a patch to submit. A single solution that works with both Electric and Fuerte would be ideal.

UPDATE: I've given some thought to how this could be done in Python. First make the import conditional:

try:
    from sensor_msgs.msg import TimeReference
except ImportError:
    TimeReference = None

Then, make sure TimeReference is True before doing anything with the gpstime variables:

if TimeReference:
    gpstimePub = rospy.Publisher('time_reference', TimeReference)
edit flag offensive delete publish link more
2

answered 2012-03-29 16:25:55 -0500

Eric Perko gravatar image Eric Perko flag of United States
6842 39 76 135
http://ericperko.com/

updated 2012-03-29 16:28:18 -0500

It would certainly be possible to remove that dependency, as long as, as @joq mentions, it's a single solution that keeps the functionality for Fuerte and provides compatibility with Electric.

Please file an enhancement ticket for this functionality on the project trac (note you can also find that link at the bottom of nmea_gps_driver under "Request features") and reply here with the link once you've done that.

If you want to go ahead and create a patch that provides Electric compatibility, just attach it to that ticket and I'll gladly take a look at it.

edit flag offensive delete publish link more

Your answer

Please start posting your answer anonymously - your answer will be saved within the current session and published after you log in or create a new account. Please try to give a substantial answer, for discussions, please use comments and please do remember to vote (after you log in)!

[hide preview]

Question tools

Follow

subscribe to rss feed

Stats

Asked: 2012-03-29 13:41:54 -0500

Seen: 112 times

Last updated: Apr 06 '12