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

adds ghost_viz package #77

Merged
merged 2 commits into from
May 30, 2024
Merged

adds ghost_viz package #77

merged 2 commits into from
May 30, 2024

Conversation

MaxxWilson
Copy link
Collaborator

@MaxxWilson MaxxWilson commented May 24, 2024

Description

Adds Trajectory Plotting for Model Predictive Control and Trajectory Generation via PlotJuggler

Reviewers

Changelog

  • Adds Trajectory Plotter for PlotJuggler
  • Adds plotjuggler and plotjuggler_ros as submodules

Reviewer Guide

  • For motion planning people, look at run instructions so you can use this
  • For devops, add these submodules to the debian pkgs list

Testing

Manual

  • ros2 launch ghost_viz trajectory_plotter.launch.py
  • Start ROS Publisher and subscribe to LabelVectorMap topic
  • Drag time series on to plot

Documentation

Current input requirements:

  • Only supports LabelVectorMap msgs as input
  • LabelVectorMap MUST contain one vector labeled "time" for plots to work
  • LabelVectorMap topic MUST be under the "/trajectory" topic namespace
  • Trajectories in LabelVectorMap msg must all be the same length (matching the number of points in time vector)

Video

PJ Test Video

adds trajectory_plotter vs pj
adds pj submodules
Copy link
Collaborator

@JakeWendling JakeWendling left a comment

Choose a reason for hiding this comment

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

lgtm

@MaxxWilson MaxxWilson merged commit 542a804 into develop May 30, 2024
1 check passed
@xwilson03
Copy link
Collaborator

xwilson03 commented Jun 2, 2024

Small update post-merge for any future viewers; plotjuggler isnt able to be debianized due to dependencies on ghost_msgs, which definitely cant be debianized because of its volatility.

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.

3 participants