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

Compression with game phase #33

Open
wants to merge 3 commits into
base: compression-example
Choose a base branch
from

Conversation

ErdoganSeref
Copy link

Extending the script for lichess-org/lila#4129. Added the method codes so the script is compatible to the Python script.

Use absolute frequency instead of relative frequency to avoid loss of precision
Make copy pasting to other source code easier
@niklasf niklasf changed the base branch from master to compression-example March 29, 2023 18:05
@niklasf
Copy link
Owner

niklasf commented Apr 9, 2023

Hi. Could it be that we need three separate code books?

@ErdoganSeref
Copy link
Author

codes.txt is the output of the script.

@niklasf
Copy link
Owner

niklasf commented Apr 10, 2023

So that's now 20.92 + 12.26 + 3.74 = 36.92 bytes per game, compared to the current 37.03. A very slight improvement.

@ErdoganSeref
Copy link
Author

I expected a bigger improvement. Is the improvement worth the added complexity?

@niklasf
Copy link
Owner

niklasf commented Apr 10, 2023

Probably no. It's disappointing, but still good to have tried this approach.

We could still play with the cutoff points, but having seen these results, I wouldn't expect much. I guess it shows that the ordering heuristic does most of the heavy lifting, and that it isn't particularly sensitive to game phase.

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.

2 participants