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

Preparation for tests implying python scripts and publishing topics. #114

Draft
wants to merge 24 commits into
base: humble
Choose a base branch
from

Conversation

olivier-stasse
Copy link
Member

This PR is proposing various commits to:

  • Remove the dependency as pointed out by @nim65s (thanks for that)
  • Extend timeout
  • Extend test_sot_loader to handle python script leading to publishing on ROS 2 topic.

The tests are working on MacOS X Sonoma but

start_run_python_script_ros_service(file_name, result);

is still commented.

Please consider it as a draft.

@olivier-stasse olivier-stasse marked this pull request as draft November 29, 2023 01:36
- The ROS Node
- A related callback group with reentrant type.
Set again 1 KHz
Use the callback group from the node python_interpreter.
The catch is to have the same node, with reentrant type
for the server and the client when both are loaded in the same process.
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.

1 participant