Skip to content

Conversation

@cmgchess
Copy link
Contributor

@cmgchess cmgchess commented Dec 23, 2023

close #14276

image
image

@ornicar ornicar merged commit e2afe34 into lichess-org:master Dec 23, 2023
kraktus added a commit to kraktus/lila that referenced this pull request Jan 6, 2024
Fixes lichess-org#14393

The issuse was created lichess-org#14277 which itself was introduced by 4a54a23. More specifically a76f4c3 and 0649bc5.

It was caused by changing to `looseH` which cannot handle a number, and which tried to re-unify 0 and null as the same value, which is not possible in this case. I rolled back to the old version.

I chose to use the original `h` function, so had to revert the `looseH` alias. It's more code but felt cleaner than aliasing h.

Tested for:
- rated
- unrated
- streak
@cmgchess cmgchess deleted the rating branch February 24, 2024 17:48
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.

The puzzles show how much I lost and gained

2 participants