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

Allow talking to pets #52030

Merged
merged 1 commit into from
Oct 4, 2021
Merged

Allow talking to pets #52030

merged 1 commit into from
Oct 4, 2021

Conversation

Ramza13
Copy link
Contributor

@Ramza13 Ramza13 commented Oct 1, 2021

Summary

none

Purpose of change

Handle second part of #51992
Allow talking to pet

Describe the solution

Describe alternatives you've considered

Doing the first part of #51992 as well. Turns out its a lot harder than this was.

Testing

Added this line to mon_cat in json and then started a cat dude.

"chat_topics":["TALK_CITY_COP_NEWS","TALK_CITY_COP"],

Additional context

@anothersimulacrum anothersimulacrum added the [C++] Changes (can be) made in C++. Previously named `Code` label Oct 1, 2021
@Entity-y
Copy link
Contributor

Entity-y commented Oct 2, 2021

So would this allow me to yell a sentence containing my pet's name to have it come to me?

@Ramza13
Copy link
Contributor Author

Ramza13 commented Oct 2, 2021

So would this allow me to yell a sentence containing my pet's name to have it come to me?

Nope

@ZhilkinSerg ZhilkinSerg merged commit 0a447f9 into CleverRaven:master Oct 4, 2021
@Ramza13 Ramza13 deleted the eoc_fixes branch October 4, 2021 16:53
@Ezsk
Copy link

Ezsk commented Nov 12, 2021

Interesting feature, but it seems to crash on TALK_DONE.

@Ramza13
Copy link
Contributor Author

Ramza13 commented Nov 13, 2021

Interesting feature, but it seems to crash on TALK_DONE.

Ha so just tested this and I'm getting a crash on starting the conversation not TALK_DONE. That said by chance my PR #52717 should fix the crash I am seeing and hopefully the issue you are having as well.

@Ramza13
Copy link
Contributor Author

Ramza13 commented Nov 14, 2021

Interesting feature, but it seems to crash on TALK_DONE.

So can you still cause a crash on TALK_DONE with the latest experimental? If so I'd love some details to fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants