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

Replaced voss::Point with voss::Pose in readme #108

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ssejrog
Copy link

@ssejrog ssejrog commented Sep 11, 2024

Copy/pasting the examples didn't work, found out it was a typo in Point vs Pose.

@Rocky14683
Copy link
Member

tbh you dont have to call the voss::Pose and it will still work. If you want to just delete all of the voss::Pose, I will merge the PR.

Copy link
Member

@Rocky14683 Rocky14683 left a comment

Choose a reason for hiding this comment

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

Fix incorrect indentation, then this will be merged.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Member

@Rocky14683 Rocky14683 left a comment

Choose a reason for hiding this comment

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

LGTM

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