autowarefoundation / ros2_socketcan

A ROS2 wrapper around Linux SocketCAN
Apache License 2.0
117 stars 60 forks source link

Added bus time #12

Closed asaba96 closed 2 years ago

asaba96 commented 2 years ago

added the ability to get the bus time for the can packet, versus using ros time when received; packs bus time as part of the can id struct

This packs the bus time in the CanId object vs returns it as a tuple. I'll let the maintainers decide which they prefer :)

Tested w. a can replay on my machine, using canreplay on a virtual can bus

@JWhitleyWork

JWhitleyWork commented 2 years ago

@mitsudome-r Can you please approve the workflow for this PR?

wep21 commented 2 years ago

@asaba96 Could you fix the linter errors?