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

Request for Comments: Small change to the BestMatch search algorithm #2092

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

takeshitakenji
Copy link

@takeshitakenji takeshitakenji commented Jan 1, 2021

While running ChatBot with data from the Fediverse, I noticed that it would often select matches that have a lower confidence if the minimum confidence was not reached. Would it make sense to select the result with the highest confidence instead?

One issue I noticed is that it would sometimes end up picking matches that lack any responses, and then would fall back to picking a random response. The end-user experience was similar in either case: a seemingly random response.

  1. Without this set of changes, when a match with a low confidence was chosen and a response was found.
  2. With this set of changes, when a match with a high confidence—but with no responses—was chosen, which resulted in a random response being chosen instead.

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.

None yet

1 participant