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

Elo Worth in King Danger #3204

Closed
wants to merge 1 commit into from
Closed

Elo Worth in King Danger #3204

wants to merge 1 commit into from

Conversation

FauziAkram
Copy link
Contributor

Adding the EloWorth for each term in King Danger.
Should be useful for simplifications, tuning patches, and new ideas.

non-functional change

Bench: 3870606
@ddugovic
Copy link

I am curious with each release whether maintainers will keep these data up to date.

@FauziAkram
Copy link
Contributor Author

@ddugovic it's similar to the elo estimation that we have in search.cpp, it's just a rough estimation, over long period of time the elo worth might vary a little bit, but we don't need to keep it constantly up to date.
Also, I predict that they will stay roughly accurate for quite some time, since we are not having many changes in the classical evaluation lately, and even when we had, king danger does not receive radical changes, just some small tweakings.

@vondele
Copy link
Member

vondele commented Nov 1, 2020

I think this is much like the Elo annotations in search. I think these are nice to have (not too fine-grained I would say), and they should only be updated every so often (like search every >1yr). They serve as a useful documentation for further development and tuning IMO.

@vondele vondele added the to be merged Will be merged shortly label Nov 2, 2020
@vondele vondele closed this in 931070b Nov 2, 2020
QueensGambit pushed a commit to QueensGambit/Fairy-Stockfish that referenced this pull request Nov 20, 2020
Adding the EloWorth for each term in King Danger.
Should be useful for simplifications, tuning patches, and new ideas.

closes official-stockfish/Stockfish#3204

non-functional change
Dantist pushed a commit to Dantist/Stockfish that referenced this pull request Dec 22, 2020
Adding the EloWorth for each term in King Danger.
Should be useful for simplifications, tuning patches, and new ideas.

closes official-stockfish#3204

non-functional change
BM123499 pushed a commit to BM123499/Stockfish that referenced this pull request Feb 22, 2021
Adding the EloWorth for each term in King Danger.
Should be useful for simplifications, tuning patches, and new ideas.

closes official-stockfish#3204

non-functional change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to be merged Will be merged shortly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants