Skip to content

Make parsePose() public again#253

Open
rhaschke wants to merge 1 commit intoros:rollingfrom
ubi-agni:parsePose
Open

Make parsePose() public again#253
rhaschke wants to merge 1 commit intoros:rollingfrom
ubi-agni:parsePose

Conversation

@rhaschke
Copy link
Copy Markdown
Contributor

parsePose is a very useful helper function that should be kept public.
Partially reverts 9a1f5b4
Fixes #245

Partially reverts 9a1f5b4

Signed-off-by: Robert Haschke <rhaschke@techfak.uni-bielefeld.de>
@rhaschke rhaschke marked this pull request as ready for review April 10, 2026 22:05
Copy link
Copy Markdown
Contributor

@saikishor saikishor left a comment

Choose a reason for hiding this comment

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

What kind of usecases are you looking at with this API?

Should we add explicit tests?

@rhaschke
Copy link
Copy Markdown
Contributor Author

What kind of usecases are you looking at with this API?

As explained in #245, I'm using parsePose() in custom sensor extensions to parse pose specs.

Should we add explicit tests?

parsePose() has been part of the API for ages. If you don't yet have tests, you should indeed consider adding some.

@saikishor
Copy link
Copy Markdown
Contributor

Sorry I missed #245

@scpeters
Copy link
Copy Markdown
Contributor

we are very near the freeze date, but we can still release this to Lyrical after the release since it is not a breaking change. apologies that it has not been merged yet

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.

Make parsePose() public again

3 participants