search

LCAS / topological_navigation

The topological navigation framework
Apache License 2.0
35 stars 37 forks source link

max_vel_trans fix for melodic #13

Closed gpdas closed 4 years ago

gpdas commented 4 years ago

max_trans_vel is replaced by max_vel_trans in melodic. As of now, this parameter is reconfigured after each edge in execute_policy_server.py, but not in navigate.py.

To make this backward compatible with kinetic, both max_trans_vel and max_vel_trans are configured for setting. However, as reconfiguration for all configured parameters are done in a single step, using try... except... might be difficult. As a temp fix, only the existing parameters will be tried to be reconfigured now.

This may fix #10

Jailander commented 4 years ago

Looks good will test in a bit and merge :)