Skip to content
This repository was archived by the owner on Nov 3, 2023. It is now read-only.

Curated response generators #4197

Merged
merged 3 commits into from
Dec 1, 2021
Merged

Curated response generators #4197

merged 3 commits into from
Dec 1, 2021

Conversation

mojtaba-komeili
Copy link
Contributor

Patch description
Adds two functions for generating canned curated responses that may replace an agent utterances. This may happen in the beginning of a conversation to initiate a good topic (generate_init_prompot function), or when the safety classifier detects an inappropriate dialogue (generate_safe_response changes the topic to something safer).

Testing steps
WILL ADD IN THE NEXT COMMIT

@mojtaba-komeili mojtaba-komeili changed the title main functions Curated response generators Nov 18, 2021
melanierk
melanierk previously approved these changes Nov 19, 2021
"tonight game.",
"next year Oscar.",
"traveling to exotic destinations.",
"read new books.",
Copy link
Contributor

Choose a reason for hiding this comment

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

"Have you heard anything about read new books."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, it may generate non-sense like that too. I will have a second look and change them for a better template.

"I was recently read about",
"I am really excited about",
"I know many people are intereseted in",
"I think it is excting to talk about",
Copy link
Contributor

Choose a reason for hiding this comment

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

typos

@stephenroller stephenroller dismissed melanierk’s stale review November 20, 2021 16:03

templates need to be fixed

@mojtaba-komeili
Copy link
Contributor Author

Thanks to @melanierk, we have an extended version of the safe responses sentences. Looking at some of them they look a lot more natural than before, although there is still change of having weird combinations, but it seems much less likely.

@mojtaba-komeili
Copy link
Contributor Author

Hey @stephenroller, could you have a second look at this one please? Thanks!

@mojtaba-komeili mojtaba-komeili merged commit 13616bd into main Dec 1, 2021
@mojtaba-komeili mojtaba-komeili deleted the curated_responses branch December 1, 2021 19:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants