Skip to content

Conversation

wentywenty
Copy link
Contributor

  - imu_transformer
  - imu_calib
  - polygon_msgs
  - polygon_utils
  - color_util
  - rclc_parameter
  - ros_humble_rclc_lifecycle
  - ros_humble_rclc_examples
  - ptz_action_server_msgs
  - py_trees

@traversaro
Copy link
Member

py-trees seems to depend on poetry-core, but there is no such key in https://github.com/ros/rosdistro/blob/0bd6f31c5cfb102d5fa65972a4883996c54bd3be/rosdep/python.yaml, so I think that is the reason why the dependency is not listed, I guess we can add the dependency via patches/dependencies.yaml. Interestingly, py-trees is also available on PyPI, so it would be a good candidate to unvendor via generate_dummy_package_with_run_deps, but it seems that it was never added to conda-forge (see conda-forge/staged-recipes#28727), so for now packaging it as a non-dummy robostack package make sense.

Co-authored-by: Silvio Traversaro <[email protected]>
@wolfv
Copy link
Member

wolfv commented Jul 30, 2025

Looks like some poetry dependency is indeed missing:

2025-07-30T09:12:49.4679351Z  │ │ pip._vendor.pyproject_hooks._impl.BackendUnavailable: Cannot import 'poetry.core.masonry.api'

@Tobias-Fischer
Copy link
Contributor

"poetry.core.masonry.api refers to the PEP 517 build backend implementation provided by the poetry-core package."

@traversaro
Copy link
Member

I added it in 66ea6b9, let's see if it works.

@traversaro
Copy link
Member

Ok, the poetry problem seems solved, there are some Windows failures but they seem to be related to something else.

@wentywenty
Copy link
Contributor Author

Sorry, I'm on my way and haven't replied to you after submitting.

@wentywenty
Copy link
Contributor Author

Next time I will try it on my own fork and test the code with action

@wentywenty
Copy link
Contributor Author

Because I need to run some decision-making and robot control content on Windows, thank you for this project, which seamlessly integrates the robot operating system on Windows. The behavior tree framework I originally ran can be highly integrated again.❤️

@traversaro
Copy link
Member

Next time I will try it on my own fork and test the code with action

No problem in using upstream PRs to iterate with CI, we do that a lot. : )

@wentywenty
Copy link
Contributor Author

To be honest, when I was training college students, I often had to learn Linux before learning ROS, which resulted in me spending most of my time teaching them how to use Linux. Thanks to this project, novices can quickly become familiar with ROS, a great operating system.❤️❤️❤️

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.

4 participants