Include option to use PointCloud Transport#761
Include option to use PointCloud Transport#761SteveMacenski merged 4 commits intoros-navigation:masterfrom
Conversation
Signed-off-by: ElSayed ElSheikh <elsayed.elsheikh97@gmail.com>
SteveMacenski
left a comment
There was a problem hiding this comment.
LGTM! Can you add in the migration guide reference for support for point cloud transport compression in the collision monitor and costmap layers?
Sure! :-) |
|
@elsayedelsheikh I think that's all that's blocking here. I'd be curious if you had any perf metrics if using compression vs not using compression to share in it :-) |
Signed-off-by: ElSayed ElSheikh <elsayed.elsheikh97@gmail.com>
Signed-off-by: ElSayed ElSheikh <elsayed.elsheikh97@gmail.com>
|
@SteveMacenski Check this migration guide. The guide currently covers bandwidth only for performance metrics as I didn't mention encoding/decoding speed, lossy vs. lossless, or CPU load. I guess if the sensor manufacturer provides |
0ff80e9 to
cac522c
Compare
Updates related to ros-navigation/navigation2#5264