Skip to content

Add mrs_lib::ServiceClient - #139

Open
pum1k wants to merge 1 commit into
pum1k/fix/coro-expected-compatibilityfrom
pum1k/feat/service-client
Open

pum1k wants to merge 1 commit into
pum1k/fix/coro-expected-compatibilityfrom
pum1k/feat/service-client

Conversation

@pum1k

@pum1k pum1k commented Sep 14, 2026

Copy link
Copy Markdown
Member

Summary

Add updated mrs_lib::ServiceClient, a rclcpp::Client wrapper with coroutines support.

This class replaces the mrs_lib::ServiceClientHanlder. Main changes:

  • construction from node interfaces (does not need node shared ptr)
  • no default constructor - prevents invalid states by requiring initialization
  • better support for coroutines
  • error messages with std::expected (instead of previous std::optional)

@pum1k
pum1k added this pull request to stack #138 September 14, 2026 09:13

This branch has not been deployed

No deployments
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