Skip to content

Feature/non delay fusion - #44

Open
ipa-svn wants to merge 334 commits into
ipa320:indigo_devfrom
ipa-svn:feature/non_delay_fusion
Open

Feature/non delay fusion#44
ipa-svn wants to merge 334 commits into
ipa320:indigo_devfrom
ipa-svn:feature/non_delay_fusion

Conversation

@ipa-svn

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

Copy link
Copy Markdown

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

mgruhler commented Jul 5, 2016

Copy link
Copy Markdown
Member

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

ipa-rmb commented Jul 5, 2016

Copy link
Copy Markdown
Contributor

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

@fmessmer

fmessmer commented Jan 19, 2017

Copy link
Copy Markdown
Member

@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
Copy Markdown
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