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

eCAL 6 Serialization / Msg API Review #1988

Open
2 of 6 tasks
KerstinKeller opened this issue Jan 31, 2025 · 0 comments · Fixed by #1964 · May be fixed by #1981
Open
2 of 6 tasks

eCAL 6 Serialization / Msg API Review #1988

KerstinKeller opened this issue Jan 31, 2025 · 0 comments · Fixed by #1964 · May be fixed by #1981
Milestone

Comments

@KerstinKeller
Copy link
Contributor

KerstinKeller commented Jan 31, 2025

The Message support (protobuf, capnproto, ...) are based on top of eCAL C++ API. We need to review this API carefully for the 6.0 release

eCAL Msg API

msg/publisher.h / msg/subscriber.h @KerstinKeller

  • Subscribers are still using old school callbacks. Provide new callbacks
  • Provide CMessagePublisher similar to CMessageSubscriber to add Serialization specific specific types a lot easier.
  • Consider error Handler for Msg Subscribers in case of serialization errors.
    (Currently, with a subscriber, the user gets no feedback if the serialization fails. No callbacks are called, and he might just think that no new data is arriving).

canproto

protobuf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants