Skip to content

Search simplification patch #39

Closed
wants to merge 2 commits into from

3 participants

@lucasart
lucasart commented Jan 1, 2013

Hello Marco,

I'm not very good with this git stuff, but I forked from your repo, and commited a patch to my branch. Test results are promising so far.

I don't know if this "pull request" thing is the way to do it: we'll see!

Lucas

lucasart added some commits Dec 31, 2012
@lucasart lucasart Merge pull request #1 from mcostalba/master
pull marco's master branch
bc85d8e
@lucasart lucasart Remove Threat Extension
Gret code simplification:
- instead do not futility prune threat refutations. allows_move() is therefore removed.
- also remove the !PvNode conditions from SEE pruning (search and qsearch)

4000 games at 50,000 nodes/move:
1085-989-1926 [51.2%] LOS=98.3%

4000 games in 10"+0.1"
756-751-2493 [50.1%] LOS=55.1%

signature = 4343046
3db23a4
@glinscott

Hi Lucas,

This looks like an interesting idea! My testing is not looking great for it so far though, at 10+0.01:

ELO: -8.83 +- 99%: 15.50 95%: 11.77
LOS: 0.27%
Wins: 424 Losses: 509 Draws: 2413

@RyanTaker

I think Marco pushed this so you should be able to close the request.

@lucasart lucasart closed this Oct 21, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.