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(core): add joint positions broadcaster and listener (cpp) #197

Open
wants to merge 7 commits into
base: dev/joint-positions
Choose a base branch
from

Conversation

domire8
Copy link
Member

@domire8 domire8 commented Feb 20, 2025

Description

This PR solves the issue by adding the two classes with required interfaces

Review guidelines

Estimated Time of Review: 15 minutes

Checklist before merging:

  • Confirm that the relevant changelog(s) are up-to-date in case of any user-facing changes

Sorry, something went wrong.

@domire8 domire8 linked an issue Feb 20, 2025 that may be closed by this pull request
Copy link
Member

@bpapaspyros bpapaspyros left a comment

Choose a reason for hiding this comment

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

small nit/question, otherwise looks good to me

@domire8 domire8 force-pushed the dev/joint-positions branch from f723af6 to 966b2eb Compare March 11, 2025 10:39
@domire8 domire8 changed the title feat(core): add joint positions broadcaster and listener feat(core): add joint positions broadcaster and listener (cpp) Mar 11, 2025
@domire8 domire8 force-pushed the dev/joint-positions branch from 966b2eb to e14c5e5 Compare March 11, 2025 11:45
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.

Implement joint positions broadcaster and listener
2 participants