intel / collision-avoidance-library

A framework for testing and benchmarking collision avoidance strategies
Apache License 2.0
81 stars 39 forks source link

coav-control: Visual Debug #126

Closed rchiossi closed 7 years ago

rchiossi commented 7 years ago

This patchset integrates vdebug functionality and depth camera visualization into coav-control. It is a compilation of the visualization tool draft implemented my @mbelluzzo and the recent vdebug rework.

This is how it looks like next to gazebo simulation: ss2

rchiossi commented 7 years ago

About your points:

I agree that we should have a better way to inform what data is expected, but I the old way was a cast nightmare. Any suggestion on how to improve it?

The naming issue sounds fair, but I guess even other depth_cameras returning uint16 might need extra fields not provided. Not sure if just changing the name is a good solution here. Maybe a generic data pointer would be better.

mbelluzzo commented 7 years ago

"If you don't mind, I'll just create the new header in a new commit.". Sure.

rchiossi commented 7 years ago

merged