-
Notifications
You must be signed in to change notification settings - Fork 23
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
Alternative methods for visualizing joint trajectories #126
Comments
I agree, and overwriting when namespace and identifier match seems like the simplest and most logical solution. |
This should now be addressed but you need port changes from a PR recently merged into tesseract_ros repository. |
Linking to the PR for reference |
On a related note, would it be difficult to visualize all individual states of the trajectory at once? That would also be a nice feature for the online demo so you could watch how all the states change as new trajectories are planned |
Closing; addressed in #131 |
When a new trajectory is received by the joint trajectory monitor, it always appends the trajectory to the visualization using the event linked below.
tesseract_ros2/tesseract_rviz/src/joint_trajectory_monitor_properties.cpp
Line 108 in 3b7e8b0
This can be problematic for displaying the results of online planning (as in the example) because the computer will quickly run out of memory stashing all the trajectories.
We should consider adding a mechanism to the existing widget to allow a trajectory (maybe with the same namespace and identifier) to be overwritten rather than appended to the list of trajectories. Alternatively, we might also think about making a display specifically for joint trajectories that doesn't stash all trajectories
The text was updated successfully, but these errors were encountered: