-
Notifications
You must be signed in to change notification settings - Fork 48
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
update setup.sh for human-dynamics profile #257
Conversation
@kouroshD is there any other path that needs to be add for the |
This is exactly the reason why I would prefer if However, this is not directly related to this PR. To finish the job, can you also:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See PR comments.
I think this was already handled with 9c6f139
|
README.md
Outdated
@@ -543,9 +542,13 @@ This profile is enabled by the `ROBOTOLOGY_ENABLE_HUMAN_DYNAMICS` CMake option. | |||
### System Dependencies | |||
To run a human dynamics estimation scenario, we need a Windows machine to install the Xsens suit SDK for getting the sensory information of the human motions from [Xsens](https://www.xsens.com/) and [ESD USB CAN driver](https://esd.eu/en/products/can-usb2) to get the FTShoes/FTSkShoes sensory information. Refer to [Xsens](#xsens) and [FTShoes/FTSkShoes](#shoes) for more information about the dependencies. | |||
|
|||
### Configuration | |||
### System Dependencies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason the System Dependencies
has been duplicated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My bad, the information in this replicated System Dependencies previously were in the Configuration section (even if they refering to system dependency). I think we can remove that part.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed da2baa1
Co-Authored-By: Silvio Traversaro <[email protected]>
Co-Authored-By: Silvio Traversaro <[email protected]>
Thanks! |
The human-dynamics profile install the
human-gazebo
models, but the path to those models need to be add toYARP_DATA_DIRS
in order to be found. For this reason I add this path in thesetup.sh