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

[respeaker_ros] Avoid AudioInfo import for backward compatibility #27

Merged
merged 2 commits into from
Aug 9, 2022

Conversation

708yamaguchi
Copy link
Collaborator

I am very sorry, but for a long time, I forgot to merge the following pull request.
708yamaguchi#4

After merging this PR, both fetch15 branch and pr1040 branch has the feature introduced in the following PR.
jsk-ros-pkg#350

@708yamaguchi
Copy link
Collaborator Author

僕のミスで長らくこの2コミットだけPR2でテストできていなかったです。申し訳ありません。。。
708yamaguchi#4 をrebaseしただけなので、マージしても大丈夫ですよね
cc @iory

@708yamaguchi 708yamaguchi merged commit 4079d98 into knorth55:pr1040 Aug 9, 2022
@708yamaguchi
Copy link
Collaborator Author

ありがとうございます!

このプルリクエストを含んだ最新のknorth55/pr1040ブランチをPR1040体内PCに持ってきました。
その後、sudo initctl stop/start robotして、サービスを再起動しました。
起動時に目立ったエラーがないこと及び/speech_to_text_googleが正しく音声認識結果を出力するのは確認しました。
これで数日様子を見ます。

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