ethz-asl / mav_trajectory_generation

Polynomial trajectory generation and optimization, especially for rotary-wing MAVs.
Apache License 2.0
528 stars 222 forks source link

Documentation neglects to mention certain installation dependencies #105

Closed nightduck closed 3 years ago

nightduck commented 4 years ago

When building according to the instructions, two issues pop up: missing cmake references to catkin_simple and yaml_cpp_catkin. There are existing issues (#79 and #102 ) that reference these problems, but both issues are closed with the simple solution to clone requisite repositories into the src directory of the catkin workspace. However, this quintessential fix is not referenced anywhere in the Readme or the wiki. Anyone who installs this repo has to go down the same rabbithole of issue reports, and it would be convenient to mention in the Readme that you have to manually clone extra repositories. (I have provided a patch to do that)

Also convenient: have them pulled automatically, see PR #104

rikba commented 3 years ago

Should be resolved in https://github.com/ethz-asl/mav_trajectory_generation/pull/116