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

Unknown capability instance: '' #45

Merged
merged 1 commit into from
Feb 8, 2014
Merged

Unknown capability instance: '' #45

merged 1 commit into from
Feb 8, 2014

Conversation

wjwwood
Copy link
Member

@wjwwood wjwwood commented Feb 8, 2014

This error message gets printed every time I run the capability server. I've seen this in a catkin workspace with several interfaces and providers, and I'm able to reproduce it on a workspace with no capabilities or providers:

jbinney@h-> rosrun capabilities capability_server 
[WARN] [WallTime: 1391825562.852679] [0.000000] No runnable Capabilities loaded.
[INFO] [WallTime: 1391825562.868363] [0.000000] Capability Server Ready
[ERROR] [WallTime: 1391825562.869098] [0.000000] Unknown capability instance: ''

It looks like on startup the following event message comes out on /capability_server/events:

header: 
  seq: 1
  stamp: 
    secs: 0
    nsecs: 0
  frame_id: ''
capability: ''
provider: ''
type: "server_ready"
pid: 0

And the capability field being an empty string confuses the event handler: https://github.com/osrf/capabilities/blob/master/src/capabilities/server.py#L476

@jonbinney
Copy link
Contributor Author

It doesn't appear to actually break anything, just an unexpected error message.

@wjwwood
Copy link
Member

wjwwood commented Feb 8, 2014

Right you are, I added that event to make the tests more robust, but ended up missing a check for it there, thanks for catching that.

Converted to a pull request.

wjwwood added a commit that referenced this pull request Feb 8, 2014
Unknown capability instance: ''
@wjwwood wjwwood merged commit dc7f626 into master Feb 8, 2014
@wjwwood wjwwood deleted the issue_45 branch February 8, 2014 03:18
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.

2 participants