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

Alternative forumla for dynamic contempt #1558

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@ceebo

ceebo commented Apr 20, 2018

Replace the forumla involving arctan with something having similar
behaviour that can be implemented using integer-only operations.

STC:
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 34781 W: 7189 L: 7093 D: 20499
http://tests.stockfishchess.org/tests/view/5ad7c95f0ebc595700526e76

LTC:
LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
Total: 39743 W: 5950 L: 5857 D: 27936
http://tests.stockfishchess.org/tests/view/5ad886ee0ebc595700526e9b

Bench: 5236139

Alternative forumla for dynamic contempt
Replace the forumla involving arctan with something having similar
behaviour that can be implemented using integer-only operations.

STC:
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 34781 W: 7189 L: 7093 D: 20499
http://tests.stockfishchess.org/tests/view/5ad7c95f0ebc595700526e76

LTC:
LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
Total: 39743 W: 5950 L: 5857 D: 27936
http://tests.stockfishchess.org/tests/view/5ad886ee0ebc595700526e9b

Bench: 5236139
@ceebo

This comment has been minimized.

Show comment
Hide comment
@ceebo

ceebo Apr 20, 2018

This was run with simplification bounds because it was suggested the formula was simpler to understand and didn't use any floating-point operations. If this is not sufficient justification then there is no problem to reject the patch.

Thanks to all for the congrats and thanks to @gonlem for the nice graph in the discussion of #1553.

ceebo commented Apr 20, 2018

This was run with simplification bounds because it was suggested the formula was simpler to understand and didn't use any floating-point operations. If this is not sufficient justification then there is no problem to reject the patch.

Thanks to all for the congrats and thanks to @gonlem for the nice graph in the discussion of #1553.

@IIvec

This comment has been minimized.

Show comment
Hide comment
@IIvec

IIvec Apr 21, 2018

I like it.

IIvec commented Apr 21, 2018

I like it.

@Stefano80

This comment has been minimized.

Show comment
Hide comment
@Stefano80

Stefano80 Apr 21, 2018

I like it too, this is a clear simplification in my opinion.

Stefano80 commented Apr 21, 2018

I like it too, this is a clear simplification in my opinion.

@snicolet snicolet closed this in aef7076 Apr 23, 2018

@snicolet

This comment has been minimized.

Show comment
Hide comment
@snicolet

snicolet Apr 23, 2018

Merged via aef7076, thanks :-)

snicolet commented Apr 23, 2018

Merged via aef7076, thanks :-)

goodkov added a commit to goodkov/Stockfish that referenced this pull request Jul 21, 2018

Alternative formula for dynamic contempt
Replace the formula involving arctan with something having similar
behaviour that can be implemented using integer-only operations.

STC:
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 34781 W: 7189 L: 7093 D: 20499
http://tests.stockfishchess.org/tests/view/5ad7c95f0ebc595700526e76

LTC:
LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
Total: 39743 W: 5950 L: 5857 D: 27936
http://tests.stockfishchess.org/tests/view/5ad886ee0ebc595700526e9b

Closes official-stockfish#1558

Bench: 5549801

aggg63 added a commit to aggg63/Stockfish that referenced this pull request Aug 22, 2018

Alternative formula for dynamic contempt
Replace the formula involving arctan with something having similar
behaviour that can be implemented using integer-only operations.

STC:
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 34781 W: 7189 L: 7093 D: 20499
http://tests.stockfishchess.org/tests/view/5ad7c95f0ebc595700526e76

LTC:
LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
Total: 39743 W: 5950 L: 5857 D: 27936
http://tests.stockfishchess.org/tests/view/5ad886ee0ebc595700526e9b

Closes official-stockfish#1558

Bench: 5549801
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment