Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/non delay fusion #44

Open
wants to merge 334 commits into
base: indigo_dev
Choose a base branch
from

Conversation

ipa-svn
Copy link

@ipa-svn ipa-svn commented Jul 2, 2016

New Features; Added packages for laser based people tracking (based on tracking of leg pairs) and people fusion.

High level model association done.
High level model association done.
Added visualization script to display the lifetime of the people tracker.
@mgruhler
Copy link
Member

mgruhler commented Jul 5, 2016

Some comments :-)

  • 330 commits, 176 changed files?!?!?!
  • travis not building due to missing add_dependencies on generated messages for (at least) the library
  • In general, please review the CMakeLists.txt. There are at least some strange include_directories calls
  • Many BSD and licenses with copyright Willow Garage, etc. Where do those files come from?
  • There seem to be some BFL files. Why not use the released version? Or include the gitlab one?

@ipa-rmb
Copy link
Contributor

ipa-rmb commented Jul 5, 2016

and please add some documentation (quick start/usage, topics in/out, etc.).

@fmessmer
Copy link
Member

fmessmer commented Jan 19, 2017

@ipa-fmw @ipa-svn
What's the status here?
Please reply to @ipa-mig and @ipa-rmb comments - even if it's just for documentation within this PR discussion...
Is this currently used somewhere?
Do we need to work on this?

@floweisshardt
Copy link
Member

this is currently not used, we stopped development/integration on cob back in summer.

Nevertheless we're still lacking this functionality and should continue in mid-term range.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants