Add HDE profile to superbuild #231
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -44,6 +44,7 @@ ycm_ep_helper(ICUB TYPE GIT | |
| -DENABLE_icubmod_canBusSkin:BOOL=${ROBOTOLOGY_ENABLE_ICUB_HEAD} | ||
| -DENABLE_icubmod_canBusVirtualAnalogSensor:BOOL=${ROBOTOLOGY_ENABLE_ICUB_HEAD} | ||
| -DENABLE_icubmod_cfw2can:BOOL=${ROBOTOLOGY_USES_CFW2CAN} | ||
| -DENABLE_icubmod_ecan:BOOL=${ROBOTOLOGY_USES_ESDCAN} | ||
| -DENABLE_icubmod_embObjFTsensor:BOOL=${ROBOTOLOGY_ENABLE_ICUB_HEAD} | ||
| -DENABLE_icubmod_embObjIMU:BOOL=${ROBOTOLOGY_ENABLE_ICUB_HEAD} | ||
| -DENABLE_icubmod_embObjInertials:BOOL=${ROBOTOLOGY_ENABLE_ICUB_HEAD} | ||
|
|
||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| # Copyright (C) 2019 iCub Facility, Istituto Italiano di Tecnologia | ||
| # CopyPolicy: Released under the terms of the LGPLv2.1 or later, see LGPL.TXT | ||
|
|
||
| include(YCMEPHelper) | ||
| include(FindOrBuildPackage) | ||
|
|
||
| find_or_build_package(YARP QUIET) | ||
|
|
||
| ycm_ep_helper(forcetorque-yarp-devices TYPE GIT | ||
| STYLE GITHUB | ||
| REPOSITORY robotology/forcetorque-yarp-devices.git | ||
| TAG master | ||
| COMPONENT iCub | ||
| FOLDER robotology | ||
| DEPENDS YARP | ||
| CMAKE_CACHE_ARGS -DENABLE_ftshoe:BOOL=ON) | ||
|
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| # Copyright (C) 2018 iCub Facility, Istituto Italiano di Tecnologia | ||
| # CopyPolicy: Released under the terms of the LGPLv2.1 or later, see LGPL.TXT | ||
|
|
||
| include(YCMEPHelper) | ||
| include(FindOrBuildPackage) | ||
|
|
||
| find_or_build_package(YARP QUIET) | ||
| find_or_build_package(iDynTree QUIET) | ||
| find_or_build_package(wearables QUIET) | ||
| find_or_build_package(osqp QUIET) | ||
| find_or_build_package(OsqpEigen QUIET) | ||
|
|
||
| ycm_ep_helper(human-dynamics-estimation TYPE GIT | ||
| STYLE GITHUB | ||
| REPOSITORY robotology/human-dynamics-estimation.git | ||
| TAG master | ||
| COMPONENT human_dynamics | ||
| FOLDER robotology | ||
| DEPENDS iDynTree | ||
| YARP | ||
| wearables | ||
| osqp | ||
| OsqpEigen) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,12 @@ | ||
| # Copyright (C) 2019 iCub Facility, Istituto Italiano di Tecnologia | ||
| # CopyPolicy: Released under the terms of the LGPLv2.1 or later, see LGPL.TXT | ||
|
|
||
| include(YCMEPHelper) | ||
| include(FindOrBuildPackage) | ||
|
|
||
| ycm_ep_helper(human-gazebo TYPE GIT | ||
| STYLE GITHUB | ||
| REPOSITORY robotology/human-gazebo.git | ||
| TAG master | ||
| COMPONENT human_dynamics | ||
| FOLDER robotology) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| # Copyright (C) 2018 iCub Facility, Istituto Italiano di Tecnologia | ||
| # CopyPolicy: Released under the terms of the LGPLv2.1 or later, see LGPL.TXT | ||
|
|
||
| include(YCMEPHelper) | ||
| include(FindOrBuildPackage) | ||
|
|
||
| find_or_build_package(YARP QUIET) | ||
| find_or_build_package(iDynTree QUIET) | ||
|
|
||
| set(WEARABLES_CMAKE_ARGS "") | ||
| if(WIN32) | ||
| list(APPEND WEARABLES_CMAKE_ARGS -DXSENS_MVN_USE_SDK:BOOL=${ROBOTOLOGY_USES_XSENS_MVN_SDK} ) | ||
| endif() | ||
kouroshD marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| ycm_ep_helper(wearables TYPE GIT | ||
| STYLE GITHUB | ||
| REPOSITORY robotology/wearables.git | ||
| TAG master | ||
| COMPONENT human_dynamics | ||
| FOLDER robotology | ||
| DEPENDS YARP | ||
| iDynTree | ||
| CMAKE_ARGS ${WEARABLES_CMAKE_ARGS}) | ||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I am not sure if we already discussed during the review, but the idea behind profiles is that they are completly independent, and if a user need to set
ROBOTOLOGY_ENABLE_ICUB_HEADtoTRUEon a given machine (for example because he need to connect to iCub CAN devices) he/she can do it manualy. If instead any project inROBOTOLOGY_ENABLE_HUMAN_DYNAMICSdepends for compilation on a given project/option, this should be added as all the other 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.
There is not the compile-time dependency. What if we set
ROBOTOLOGY_ENABLE_ICUB_HEADtoTRUEandfind_or_build_package(forcetorque-yarp-devices), ifROBOTOLOGY_USES_ESDCANisTRUE? Since with this option the user is indicating, he/she wants to use FT sensors.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.
@traversaro