search

strands-project / strands_perception_people

long-term detection, tracking and recognition of people
96 stars 70 forks source link

Removing bayestracking submodule #88

Closed cdondrup closed 9 years ago

cdondrup commented 10 years ago

As the bayestracking library is now available from our ppa server (apt-get install ros-hydro-bayes-tracking) and included in our rosdep definitions (as bayes_tracking) it is no longer needed as a submodule and the branch it is cloned from is deprecated.

If the local environment has been set-up correctly, following the guide by @marc-hanheide, it will be installed via rosdep.

_This PR requires that people set-up their local environments using the correct ppa and rosdep definitions. Otherwise the build will fail._

marc-hanheide commented 10 years ago

So, it seems to work, but I'm not sure if we are confident enough to merge this already now... If people install manually using sudo apt-get install ros-hydro-bayes-tracking then they wouldn't need the local environment setup, correct?

cdondrup commented 10 years ago

yes, you can also add the ppa as you described here: https://github.com/strands-project-releases/strands-releases/wiki and use apt-get. That's all that rosdep does anyway.