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

Add description parameter to plotTrajectory functions #94

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

rjoomen
Copy link
Contributor

@rjoomen rjoomen commented Feb 20, 2024

  • Add description parameter to plotTrajectory functions
  • Add a plotTrajectories function to plot multiple trajectories in one set
  • Minor API change: removed parameter ns from one function, as it was unused but might give the false impression it would be used instead of traj.ns
  • Fixed the clear() function, as it removed all markers except the tool markers

- Add a plotTrajectories function to plot multiple trajectories in one set
- Minor API change: removed parameter ns from one function, as it was unused but might give the false impression it would be used instead of traj.ns
- Fixed the clear() function, as it removed all markers except the tool markers
@Levi-Armstrong
Copy link
Contributor

Looks like there are a few clang-tidy errors

@rjoomen
Copy link
Contributor Author

rjoomen commented Feb 22, 2024

Looks like there are a few clang-tidy errors

That's right, Tesseract is full of clang-tidy errors with recent clang-tidy versions. They are not in the parts I changed here, though, I too care of that already. I'll start an effort correcting them throughout tesseract_ros2 soon, but for now I'll leave them, as it's a lot of work to correct them all.

@rjoomen rjoomen merged commit c37a56f into tesseract-robotics:master Feb 22, 2024
7 of 8 checks passed
@rjoomen rjoomen deleted the trajectoryfix branch February 23, 2024 05:49
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