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

test google_chat_ros #392

Merged
merged 59 commits into from
Nov 12, 2022
Merged

test google_chat_ros #392

merged 59 commits into from
Nov 12, 2022

Conversation

k-okada
Copy link
Member

@k-okada k-okada commented Nov 2, 2022

No description provided.

mqcmd196 and others added 30 commits June 28, 2022 10:13
add actionmsg, catkin install, ros action

Add action server for google chat ros

rename action msg, edit more info to readme

add figs to readme and fix style

Add roseus client

add euslisp sample to readme

add link for jsk internal users

use dummy spacename

Add some comments and fix typo

key args to euslisp

fix with flake8

fix README as review comment

set default type name

fix CMake install directory

rename action name
add space activity msg

fix message names, add more units

add rosparam in launch

using request, not flask

fix so many bugs and works fine with dm, still bugs with space

supress SSL reset error and add header debug

fix unicode bugs, set default dic value
Add and fix message type for supporting card type

support card type, TODO: uploader
fix upload service

fix name bug, remove upload parent dir
fix botton key value json bug

fix value error when get card activity and parse key value
…ogflow, soundplay

[google_chat_ros] chmod +x helper.py
[google_chat_ros][WIP] pub sub mode
…ng json, add response from Google Chat API server to sendmessage action result
@k-okada k-okada force-pushed the google_chat_ros branch 3 times, most recently from 940ba15 to 4130fac Compare November 8, 2022 12:18
@k-okada k-okada force-pushed the google_chat_ros branch 2 times, most recently from 9964f54 to 9f75465 Compare November 9, 2022 01:15
2022-11-08T13:11:12.5532139Z [google_chat_ros:make]   File "/github/home/ros/ws_jsk_3rdparty/build/google_chat_ros/venv/bin/pip-compile", line 7, in <module>
2022-11-08T13:11:12.5532848Z [google_chat_ros:make]     from piptools.scripts.compile import cli
2022-11-08T13:11:12.5533901Z [google_chat_ros:make]   File "/github/home/ros/ws_jsk_3rdparty/build/google_chat_ros/venv/lib/python3.5/site-packages/piptools/__init__.py", line 3, in <module>
2022-11-08T13:11:12.5534577Z [google_chat_ros:make]     from piptools.click import secho
2022-11-08T13:11:12.5535391Z [google_chat_ros:make]   File "/github/home/ros/ws_jsk_3rdparty/build/google_chat_ros/venv/lib/python3.5/site-packages/piptools/click.py", line 3, in <module>
2022-11-08T13:11:12.5536000Z [google_chat_ros:make]     import click
2022-11-08T13:11:12.5536768Z [google_chat_ros:make]   File "/github/home/ros/ws_jsk_3rdparty/build/google_chat_ros/venv/lib/python3.5/site-packages/click/__init__.py", line 7, in <module>
2022-11-08T13:11:12.5537421Z [google_chat_ros:make]     from .core import Argument as Argument
2022-11-08T13:11:12.5538281Z [google_chat_ros:make]   File "/github/home/ros/ws_jsk_3rdparty/build/google_chat_ros/venv/lib/python3.5/site-packages/click/core.py", line 83
2022-11-08T13:11:12.5538949Z [google_chat_ros:make]     f"{hint}. Command {base_command.name!r} is set to chain and"
2022-11-08T13:11:12.5539501Z [google_chat_ros:make]                                                                ^
2022-11-08T13:11:12.5540023Z [google_chat_ros:make] SyntaxError: invalid syntax

we need to downgrade piptools/click or use newer python3
@mqcmd196
Copy link
Member

Thank you for working around the test. So I should have passed environment vars to cmake like dbe9fdd . Should we update jsk_travis?

@k-okada k-okada merged commit a8464dc into jsk-ros-pkg:master Nov 12, 2022
@k-okada k-okada deleted the google_chat_ros branch November 12, 2022 05:37
@k-okada
Copy link
Member Author

k-okada commented Nov 12, 2022

@mqcmd196 you do not need to update jsk_travis until we have more cases that requires --make-args.

There are two picutres that we need to always keep in mind.

  1. https://community.robotshop.com/blog/show/robot-humor-dinner-comics
  2. https://youtu.be/vRKrJNfvFU4?t=896
    and in my opition, is not time to focus on jsk_travis.

@k-okada
Copy link
Member Author

k-okada commented Apr 5, 2023

from @mqcmd196

process[google_chat_ros_helper-3]: started with pid [16530]
[INFO] [1680417167.311238]: [/google_chat_ros] Starting Google Chat REST service...
[INFO] [1680417167.319403]: [/google_chat_ros] Succeeded in starting Google Chat REST service
[INFO] [1680417167.337634]: [/google_chat_ros] Expected to use Google Cloud Pub Sub service
Traceback (most recent call last):
  File "/home/k-okada/catkin_ws/ws_3rdparty/src/jsk_3rdparty/google_chat_ros/scripts/google_chat_ros_node.py", line 473, in <module>
    node = GoogleChatROS()
  File "/home/k-okada/catkin_ws/ws_3rdparty/src/jsk_3rdparty/google_chat_ros/scripts/google_chat_ros_node.py", line 80,
in __init__
    self._pubsub_client.run()
  File "/home/k-okada/catkin_ws/ws_3rdparty/src/jsk_3rdparty/google_chat_ros/src/google_chat_ros/google_chat.py", line 135, in run
    self._streaming_pull_future.result()
  File "/usr/lib/python3.6/concurrent/futures/_base.py", line 432, in result
    return self.__get_result()
  File "/usr/lib/python3.6/concurrent/futures/_base.py", line 384, in __get_result
    raise self._exception
google.api_core.exceptions.NotFound: 404 Resource not found (resource=chat-sub).
これreadmeに書いてなかったんですけど,デフォルトでGoogle Pub/Subは1週間でsub側のサービスが無効化されます.なのでコンソールで再度有効化してあげる必要があります

google_chat_ros_404_error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants