Skip to content

Conversation

@iche033
Copy link
Contributor

@iche033 iche033 commented Mar 27, 2025

🦟 Bug fix

Summary

The ability to plot poses was added in #1607 for Fortress (ign-gazebo6). It seems like this feature was not forward ported to newer versions. This PR adds the feature back. When you expand the Pose item in the Component Inspector GUI plugin, you should now see the plot icon next to each pose attribute.

To test

  1. Open Plotting GUI Plugin
  2. Click on any model in scene
  3. Go to Component Inspector, expand Pose
  4. Plot any one of the pose attribute by dragging it to the plot
  5. Hit Play to see it being plotted

pose_plotting

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Signed-off-by: Ian Chen <ichen@openrobotics.org>
@iche033 iche033 requested a review from mjcarroll as a code owner March 27, 2025 23:23
@github-actions github-actions bot added the 🪵 jetty Gazebo Jetty label Mar 27, 2025
@iche033 iche033 changed the title Bring pack feature for plotting pose attributes from Component Inspector Bring back feature for plotting pose attributes from Component Inspector Mar 27, 2025
Copy link
Contributor

@arjo129 arjo129 left a comment

Choose a reason for hiding this comment

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

Ah that is where it went. ☠️

Thanks for noticing this.

@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Apr 7, 2025
@arjo129 arjo129 enabled auto-merge (squash) April 7, 2025 09:37
@iche033 iche033 disabled auto-merge April 7, 2025 17:26
@iche033 iche033 merged commit 6135ae8 into main Apr 7, 2025
8 of 9 checks passed
@iche033 iche033 deleted the pose_plotting branch April 7, 2025 17:26
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Apr 7, 2025
@iche033
Copy link
Contributor Author

iche033 commented Apr 7, 2025

@Mergifyio backport gz-sim8 gz-sim9

@mergify
Copy link
Contributor

mergify bot commented Apr 7, 2025

backport gz-sim8 gz-sim9

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Apr 7, 2025
…tor (#2835)

Signed-off-by: Ian Chen <ichen@openrobotics.org>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit 6135ae8)
mergify bot pushed a commit that referenced this pull request Apr 7, 2025
…tor (#2835)

Signed-off-by: Ian Chen <ichen@openrobotics.org>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit 6135ae8)
iche033 added a commit that referenced this pull request Apr 8, 2025
…tor (#2835)

Signed-off-by: Ian Chen <ichen@openrobotics.org>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit 6135ae8)
iche033 added a commit that referenced this pull request Apr 8, 2025
…tor (#2835)

Signed-off-by: Ian Chen <ichen@openrobotics.org>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit 6135ae8)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🪵 jetty Gazebo Jetty

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants