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

parameter_blackboard node parameter cannot be set because it was not declared #851

Closed
fujitatomoya opened this issue Sep 13, 2019 · 3 comments
Labels
bug Something isn't working

Comments

@fujitatomoya
Copy link
Collaborator

Bug report

Required Info:

  • Operating System:
    • Ubuntu18.04
  • Installation type:
    • Source
  • Version or commit hash:
  • DDS implementation:
    • Fast-RTPS
  • Client library (if applicable):
    • rclcpp, demo_nodes_cpp

Steps to reproduce issue

Terminal-1:

root@6cc2c62d0dd0:~/ros2_colcon_ws# ros2 run demo_nodes_cpp parameter_blackboard
[INFO] [parameter_blackboard]: Parameter blackboard node named '/parameter_blackboard' ready, and serving '1' parameters already!
[WARN] [rclcpp]: Failed to set parameter: parameter 'foo' cannot be set because it was not declared

Terminal-2:

root@6cc2c62d0dd0:~/ros2_colcon_ws# ros2 param set /parameter_blackboard foo 1
Setting parameter failed: parameter 'foo' cannot be set because it was not declared

Expected behavior

parameter should be set in success.

Actual behavior

parameter does not get set cz parameter is undeclared.

Additional information

parameter_blackboard specifies NodeOption allow_undeclare_parameter, so it should be able to set parameter without error.

the following fix might be related,

b214324

Feature request

W.I.P

Feature description

N.A

Implementation considerations

N.A

@ivanpauno ivanpauno added the bug Something isn't working label Sep 13, 2019
@ivanpauno
Copy link
Member

I've been able to reproduce the issue

@fujitatomoya
Copy link
Collaborator Author

ros2/demos#389 can fix this problem.

@fujitatomoya
Copy link
Collaborator Author

@ivanpauno

this can be closed.

@nuclearsandwich nuclearsandwich removed this from Proposed in Dashing Patch Release 4 Oct 16, 2019
DensoADAS pushed a commit to DensoADAS/rclcpp that referenced this issue Aug 5, 2022
* Add snapshot mode parameter to record verb


Signed-off-by: Cameron Miller <cammlle@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Development

No branches or pull requests

2 participants