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

feat: Initial implementation of ROS to OPC UA method calls #43

Merged
merged 7 commits into from
Sep 27, 2024

Conversation

philipp-caspers
Copy link
Member

@philipp-caspers philipp-caspers commented Sep 17, 2024

Adds the framework for the routing of ROS2 service calls to OPC UA methods.
One service is implemented as an example.

Closes: RR-840

@philipp-caspers philipp-caspers force-pushed the feat/add-service-cb branch 7 times, most recently from 3429a96 to c3162ff Compare September 24, 2024 07:32
Copy link
Member

@AiyionPrime AiyionPrime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"ci: Separate unit and integration tests" commitmessage

src/opc_ua_client.rs Outdated Show resolved Hide resolved
src/opc_ua_client.rs Outdated Show resolved Hide resolved
src/main.rs Outdated Show resolved Hide resolved
src/main.rs Outdated Show resolved Hide resolved
src/opc_ua_client.rs Outdated Show resolved Hide resolved
tests/test_bridge.rs Outdated Show resolved Hide resolved
tests/test_bridge.rs Outdated Show resolved Hide resolved
tests/test_bridge.rs Outdated Show resolved Hide resolved
in order to avoid confusion and better reflect the responsibility
as in the near future there will be more functionality than only
subscribing.
as a demo case for future implementation.
such that the execution of the OPC UA client loop does not block the ROS
node.
also run the unit test in sequence for now because they share the same
test resources e.g. ports or ros topics.
src/main.rs Show resolved Hide resolved
@philipp-caspers philipp-caspers merged commit 049e512 into main Sep 27, 2024
6 checks passed
@philipp-caspers philipp-caspers deleted the feat/add-service-cb branch September 27, 2024 11:49
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.

2 participants