Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Simplification: use Arctan for the optimism S-curve
This was an idea of Gontran Lemaire (gonlem), but the graphs he published did not seem accurate to me. I did my own graphs, got my own constants and here is the result: STC: LLR: 2.95 (-2.94,2.94) [-3.00,1.00] Total: 42179 W: 8704 L: 8622 D: 24853 http://tests.stockfishchess.org/tests/view/5a9db6270ebc590297cb611b LTC: LLR: 2.96 (-2.94,2.94) [-3.00,1.00] Total: 17737 W: 2702 L: 2577 D: 12458 http://tests.stockfishchess.org/tests/view/5a9eb5b20ebc590297cb61b2 Note: we are now back to 70% draw rate in selfplay mode! Bench: 5544908
- Loading branch information
82697f1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After this patch Im having losses with SF dev that didnt have before. Hope it is just a fluke. BTW I look the *note with some concerns about draw rate back to 70%. This cant be a good thing.
82697f1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't worry too much, as the ongoing regression test against SF9 and the previous one have respectively 73.9% and 73.5% draw, i.e. same drawrate within margin of error. I think the 70% from the note is at STC, and against different opponents, i don't think it is very significative.
82697f1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Current tests (regression tests) show that this patch is increasing loss rate.
82697f1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
arctan ~ A/(1+abs(A))