search

LCAS / topological_navigation

The topological navigation framework
Apache License 2.0
31 stars 36 forks source link

pip dependency #173

Closed marc-hanheide closed 7 months ago

marc-hanheide commented 7 months ago

We should avoid pip dependencies, to facilitate releases of .deb packages. I just found

https://github.com/LCAS/topological_navigation/blob/a2a1a775cbe57f72c3f73d8032b0c4edc3de1c47/topological_navigation/package.xml#L35

@Iranaphor is this required or can it be removed (blame shows you added it)?

In general, please everybody in @LCAS/aoc_devs avoid pip dependencies as much as possible. If there are any needed, let me know and we discuss ooptions.

Iranaphor commented 7 months ago

I think this was included in error.

marc-hanheide commented 7 months ago

so, can you make PR to remove?

Iranaphor commented 7 months ago

CI in progress for #174