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

Kinetic #26

Open
wants to merge 2 commits into
base: kinetic-devel
Choose a base branch
from
Open

Conversation

badaolaziang
Copy link

@badaolaziang badaolaziang commented Jan 11, 2021

Adding a kinetic branch for ubuntu 16.04, referenced with teb_local_planner kinetic-devel.

recorded link: https://github.com/badaolaziang/mpc_local_planner_kinetic

necessary dependence downloading:
sudo apt install coinor-libipopt-dev
git clone https://github.com/rst-tu-dortmund/control_box_rst.git
cd control_box_rst/
mkdir build
cd build
cmake ..
make -j8 -l8
sudo make install

@HappySamuel
Copy link

Thanks to @badaolaziang . Now i finally can catkin_make mpc_local_planner successfully on the ROS Kinetic.
#27

Best,
Samuel

@amakarow amakarow changed the base branch from melodic-devel to kinetic-devel March 8, 2021 16:05
@amakarow
Copy link
Collaborator

amakarow commented Mar 8, 2021

Thanks for this backport, I just checked the code and saw you pushed the mpc_local_planner_ros header and source file into the meta package folder/root folder. Is this intended? What is the reason for this?

@badaolaziang
Copy link
Author

Thank you for your time and comment. @amakarow . For kinetic devel version, I changed several functions to make it work in navigation-kinetic-devel version. As far as I'm concerned, I didnot intend to , and didnot change the structure of mpc_local_planner code. Can you further point it out, so I can see what I can do for this kinetic-devel version. Sorry and thank you very much for you time. Best wishes~

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.

3 participants