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
Added support for connecting to local broker #5
Conversation
hi, oyu need to rebase to get that PR valid, thx. |
By doing this the nodelet can connect to a local broker allowing direct function calls and memory access. Note that the nodelet will prefer the local broker over one specified on the command line or using ros parameters.
Rebased. |
@@ -112,6 +112,12 @@ namespace naoqicamera_driver | |||
diagnostic_updater::TimeStampStatusParam()) | |||
{ | |||
getNaoqiParams(priv_nh); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't we try to connect to the NAOqi with the parameters first ? (and if that fails try the local one)
@jondy276 , shouldn't we try to connect to the NAOqi with the parameters first ? (and if that fails try the local one) |
ping ? |
Sorry, lot of things to do. |
Allso, I have a prototype naoqi module that allows nodelets to be loaded. |
lg2m. We have nothing to load nodelets in a naoqi module so sure, make it more fun :) |
Added support for connecting to local broker
By doing this the nodelet can connect to a local broker allowing
direct function calls and memory access.
Note that the nodelet will prefer the local broker over one
specified on the command line or using ros parameters.