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

Ignore charactors which AquesTalk2 cannot recognize and update README.md #257

Merged
merged 8 commits into from
Jun 9, 2021

Conversation

708yamaguchi
Copy link
Member

Please see #255

With this pull request, for example,

’こ”^^ん/に*|ち**|は;:;:『』!!##$$&&~〜()()__`\_\\\_・・<\>*`><++@@ー==

is converted to

こんにちはー

@ayfujii
Copy link
Contributor

ayfujii commented Jun 4, 2021

#255 (comment)
に関するプルリクです。
Chaplusに「おはよう」と話しかけたときの応答例です。
置換あり:sample1, sample2
置換なし:sample1, sample2

また、小文字の「ぁ」などもエラーになることに気がついたので追記しています。

@708yamaguchi
Copy link
Member Author

708yamaguchi commented Jun 4, 2021

ありがとうございます。
「!」や「!」、「〜」を置換することで、Chaplusの返答が自然に聞こえるのが確認できました。

一方で小文字の「ぁ」などについてですが、僕がREADMEで書いているように「ぁ」から始まる文章は読めないもの「ふぁ」などの文章は読めることを確認しています。
https://github.com/jsk-ros-pkg/jsk_3rdparty/pull/257/files#diff-8551d08289cb84dd3055589b0aa6fd1b16185fdfe32427750dda1add85216797R54

なので、「ぁ」などが文頭に来たときだけ消去するようにしました。

$ echo "、ぁぁぁふぁ" | sed -e "s/^[ぁぃぅぇぉァィゥェォ、。ー]*//"
ふぁ

追記:
「たぁ」は読めない、と教えていただいたので、その旨をREADMEに書いておきました。

@k-okada k-okada merged commit e054514 into jsk-ros-pkg:master Jun 9, 2021
@708yamaguchi 708yamaguchi deleted the aques-talk-ignore branch November 24, 2021 17:43
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.

3 participants