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

[jsk_robot_startup] add google_chat function in smach_to_mail.py #72

Closed
wants to merge 133 commits into from

Conversation

mqcmd196
Copy link

related to jsk-ros-pkg#1558
I added google_chat_ros actionlib client in smach_to_mail.py .

ssk-yoshimura and others added 30 commits November 4, 2021 23:37
Assumes that following commands are executed on COBOTTA:
$ roslaunch denso_cobotta_driver denso_cobotta_driver.launch enable_calset:=false
$ roslaunch denso_cobotta_moveit_config planning_context.launch load_robot_description:=true gripper_type:=parallel
$ roslaunch denso_cobotta_control denso_cobotta_control.launch

https://github.com/DENSORobot/denso_cobotta_ros/blob/bb60e75adb8477ed3402561b4ec3ba687af3f397/denso_cobotta_bringup/launch/denso_cobotta_bringup.launch is exaggerated because it launches MoveIt!
Now 'roslaunch jsk_cobotta_startup cobotta_oss_bringup.launch' is enough for euslisp interface
kinetic: --inorder is not default
melodic and upper: --inorder is default
cf. http://wiki.ros.org/xacro#Processing_Order
@mqcmd196
Copy link
Author

Sorry for being late, I confirmed it works like this
chat_notification

@mqcmd196
Copy link
Author

@tkmtnt7000 I see, I'll update the doc.

@mqcmd196
Copy link
Author

I updated the README in 9d38196

@mqcmd196
Copy link
Author

sorry I missed merging upstream, please wait for fixing...

[jsk_robot_startup] fix typo in active_user.l
[jsk_baxter_startup] fix typo: default -> value
@mqcmd196
Copy link
Author

updated, it's ready for merged

Copy link
Owner

@k-okada k-okada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mqcmd196 thank you
pllease add jsk_robot_startup/launch/smach_to_mail.launch and accespt use_mail use_twitter, use_google_chat argumetn, also update doc/

jsk_robot_common/jsk_robot_startup/package.xml Outdated Show resolved Hide resolved
@mqcmd196
Copy link
Author

I rebased origin/master, so close this PR.

@mqcmd196 mqcmd196 closed this Aug 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.