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

Enable python bindings compilation for human-dynamics-estimation if ROBOTOLOGY_USES_PYTHON is enabled #1717

Merged
merged 4 commits into from
Sep 26, 2024

Conversation

traversaro
Copy link
Member

@traversaro traversaro changed the title Enable python bindings compilation for human-dynamics-estimation Enable python bindings compilation for human-dynamics-estimation if ROBOTOLOGY_USES_PYTHON is enabled Sep 18, 2024
@traversaro
Copy link
Member Author

Error:

-- Package robot-log-visualizer not found. Will be downloaded and built.
-- Package resolve-robotics-uri-py not found. Will be downloaded and built.
CMake Error at cmake/BuildHumanDynamicsEstimation.cmake:21:
  Parse error.  Expected a command name, got unquoted argument with text
  "//".
Call Stack (most recent call first):

@traversaro
Copy link
Member Author

Error:

-- Package robot-log-visualizer not found. Will be downloaded and built.
-- Package resolve-robotics-uri-py not found. Will be downloaded and built.
CMake Error at cmake/BuildHumanDynamicsEstimation.cmake:21:
  Parse error.  Expected a command name, got unquoted argument with text
  "//".
Call Stack (most recent call first):

Great, I was using C++ comments in CMake. -_-

@traversaro
Copy link
Member Author

This is failing with:

WARNING: Error parsing dependencies of hde: Invalid version: ''

@traversaro
Copy link
Member Author

There is indeed a bug in CMake, that will be fixed in robotology/human-dynamics-estimation#398 .

@traversaro traversaro merged commit adf3f1f into master Sep 26, 2024
29 checks passed
@traversaro traversaro deleted the fix1707 branch September 26, 2024 11:42
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.

Enable human-dynamics-estimation python bindings if ROBOTOLOGY_USES_PYTHON is enabled
1 participant