Skip to content
This repository was archived by the owner on Feb 3, 2025. It is now read-only.

Read controller/joint values in joint control widget #295

Closed
osrf-migration opened this issue Dec 10, 2012 · 13 comments
Closed

Read controller/joint values in joint control widget #295

osrf-migration opened this issue Dec 10, 2012 · 13 comments
Labels
all bug Something isn't working common::PID minor

Comments

@osrf-migration
Copy link

Original report (archived issue) by Thomas Koletschka (Bitbucket: thomasko).


Currently Gazebo's joint control widget always displays default values but doesn't read the controller's/joint's values. It also doesn't keep the values you set when you deselect and select the model again but rather goes back to displaying its default values.

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


  • set assignee_account_id to "557058:5de38267-b118-494c-aa76-4fab35448816"
  • changed assignee from "None (Bitbucket: None)" to "scpeters (Bitbucket: scpeters, GitHub: scpeters)"

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


  • changed state from "new" to "on hold"

Until 2.0

@osrf-migration
Copy link
Author

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).


  • changed state from "on hold" to "open"

@osrf-migration
Copy link
Author

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).


  • changed priority from "major" to "minor"

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


  • set component to "gui"

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


@chapulina I started working on this long ago, and I heard you mention it the other day.

@osrf-migration
Copy link
Author

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


Thanks for pointing me to this issue, I think there's a lot of room for improvement on the joint control widget. I'll take a look at your commits when I have some time.

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


It was basically just adding an extra publisher/subscriber pair. The physics::JointController got a new publisher with current gain values, and the JointControlWidget got a subscriber.

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


  • changed component from "gui" to "common::PID"

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


  • set version to "all"

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


@pchorak this might be an interesting issue to look at

@osrf-migration
Copy link
Author

Original comment by Peter Horak (Bitbucket: pchorak).


@scpeters I added a service to fix this (pull request #2730)

@osrf-migration
Copy link
Author

Original comment by Peter Horak (Bitbucket: pchorak).


  • changed state from "open" to "resolved"

A service was added in pull request #2730 (merged) so that the joint control widget can query the current joint controller values when it switches between models.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
all bug Something isn't working common::PID minor
Projects
None yet
Development

No branches or pull requests

1 participant