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

Update client.py #171

Open
wants to merge 1 commit into
base: noetic-devel
Choose a base branch
from
Open

Conversation

augustinmanecy
Copy link

@augustinmanecy augustinmanecy commented Mar 3, 2021

When using rqt_reconfigure with node with a lot of parameters, it seems some parameters are most of time not found when trying to update the configuration.
The parameters are listed in rqt_dynamic_reconfigure, but when trying to change value of some parameters "update_configuration" raises the exception line 185:

error updating parameters: don't know parameter xxx

When equivalent commands set and get succeed each time:

rosrun dynparam get <node_name> <param> 
rosrun dynparam set <node_name> <param> <value>

Not sure of what exactly happen as it is not fully repeatable, but the solution proposed in issue #163 seems to do the job.

When using rqt_reconfigure with node with a lot of parameters, it seems some parameters are most of time not found when trying to update the configuration.
The parameters are listed in rqt_dynamic_reconfigure, but when trying to change value of some parameters "update_configuration" raise the exception line 185:
```bash
error updating parameters: don't know parameter xxx
``` 
rqt_reconfigure returns:
```bash
```
When equivalent commands set and get succed each time:
```bash
rosrun dynparam get <node_name> <param> 
rosrun dynparam set <node_name> <param> <value>
```
Not sure of what exactly happen as it is not fully repeatable, but the solution proposed in issue ros#163 seems to do the job.
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.

1 participant