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

🐛 fix feedback when guesses have the same letter multiple times #8

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nanuxbe
Copy link

@nanuxbe nanuxbe commented Feb 19, 2022

This PR fixes the feedback given by the competition engine compared to the actual wordle when dealing with guesses that have the same letter(s) multiple times.

Here is an example:
The word to guess is shine and the guess is sense:

  • wordle answers with cnpnc
  • main answers with cpppc

Sideeffects of the PR:

  • Unfortunately this fix seems to break most AI's (except Bruugle and Monkey)
  • Validation of similar words in hard mode is not fixed (since it depends on LetterPopularityAI which is broken by the PR)

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.

1 participant