intel / collision-avoidance-library

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

Build: samples #43

Closed mbelluzzo closed 7 years ago

mbelluzzo commented 7 years ago

After the cmake refactoring, samples aren't being built, at least for realsense=on gazebo=on

rchiossi commented 7 years ago

Just did a clean build with the following: cmake -DWITH_REALSENSE=ON -DWITH_GAZEBO=ON ..

Samples were built as expected.

Can you provide some more info?

mbelluzzo commented 7 years ago

We were trying to run coav_gcs on the drone, so we were using gazebo = off. I did some patches to compile it without gazebo, will send Monday for review. I think you can ignore this until there.

On Thu, Nov 24, 2016, 08:32 Rodrigo Chiossi notifications@github.com wrote:

Just did a clean build with the following: cmake -DWITH_REALSENSE=ON -DWITH_GAZEBO=ON ..

Samples were built as expected.

Can you provide some more info?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/01org/collision-avoidance-library/issues/43#issuecomment-262813375, or mute the thread https://github.com/notifications/unsubscribe-auth/ABgsZbUF1l4pbRnwR2Hsl_E8idf1aIQ4ks5rBbw4gaJpZM4K2-5j .

rchiossi commented 7 years ago

The samples are not supposed to compile if gazebo is off, but thats not what you said in the issue description. coav_gcs is a bit of a mess atm, the plan is to replace it with something more generic.

mbelluzzo commented 7 years ago

Agree that it needs work. Until that, to have us moving forward and testing on the actual drone, see if those make sense for you: #44