-
Notifications
You must be signed in to change notification settings - Fork 97
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
add sample launch code for database_talker #1792
base: master
Are you sure you want to change the base?
Conversation
c882edb
to
e92bf92
Compare
<!-- openai_ros --> | ||
<group ns="openai"> | ||
<node pkg="openai_ros" type="openai_node.py" name="openai" output="screen"> | ||
<rosparam command="load" file="$(arg openai_credentials_json)" /> | ||
</node> | ||
</group> |
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.
Just a comment.
I think it may be good to put sample launch files in jsk_robot, but I am also worried about an additional dependency that is not managed by jsk-ros-pkg.
<include file="$(find dialogflow_task_executive)/launch/dialogflow_ros.launch"> | ||
<arg name="credential" value="$(arg google_credentials_json)" /> | ||
<arg name="project_id" value="collaborative-remembering-kwit" /> | ||
<arg name="enable_hotword" value="false" /> |
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.
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.
When related PRs in jsk_recognition and jsk_3rdparty merge, we can merge this
…/lifelog.launch need source install of jsk_perception and jsk_3rdparty
484de1f
to
f0c1857
Compare
…elet_manager, start_mongodb
need
https://drive.google.com/file/d/1uctY9KmmQCFbHjRF-1LXDpBSDFy8AXL1/view?usp=share_link