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.

Sign up for GitHub

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

Jump to bottom

Missing dependency in the documentation #102

Open
lrapetti opened this issue Feb 4, 2022 · 3 comments
Open

Missing dependency in the documentation #102

lrapetti opened this issue Feb 4, 2022 · 3 comments

Comments

@lrapetti
Copy link
Member

lrapetti commented Feb 4, 2022

In the repo documentation (https://github.com/robotology/walking-controllers#page_facing_up-dependencies), icub-main is not mentioned, but it seems to be a required dependency.

cc @SimoneMic feel free to add here any detail
@SimoneMic
Copy link

Yeah everything fine, did all the installation in https://icub-tech-iit.github.io/documentation/sw_installation/linux_from_sources_manual/

@lrapetti
Copy link
Member Author

The dependency on whole-body-estimators is missing as well in case you want to run the simulation example (https://github.com/robotology/walking-controllers#computer-how-to-run-the-simulation), we could specify it in the documentation (also icub-models would be needed for running the example with iCub).

@lrapetti lrapetti changed the title Missing dependency on icub-main in the documentation Missing dependency in the documentation Feb 10, 2022
@GiulioRomualdi
Copy link
Member

Hi @lrapetti and @SimoneMic, feel free to open a PR

@lrapetti lrapetti mentioned this issue Feb 18, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@GiulioRomualdi @lrapetti @SimoneMic