|
|
|
|
|
by waterproof
1600 days ago
|
|
Right, I wonder how many of the “fake/invalid” tweets that OP observed are actually this bug in the analysis code. EDIT: actually it looks like it’s correct - evaluate_guess_char() only returns “M” if there’s an instance of the guess letter that’s not accounted for. |
|
It filters out cases where the corresponding character in the answer is correct (a 'Y'), but not cases where it's used in another maybe (a 'M'). The latter requires keeping track of state in a way that this doesn't.
For example:
Which is wrong, as stated above. Which is right, even though we only changed the middle letter of the guess, not either of the broken letters. In this case the filtering works correctly.