Skip to content
Permalink
Browse files

Update continuation history for quiet move which failed high at LMR d…

…ependent on the reaearch result but use only half bonus. Bench: 3454998
  • Loading branch information...
Stefan Geschwentner
Stefan Geschwentner committed Jul 11, 2019
1 parent 2e3419b commit 1b406fb72770ec782d6015158ead3bb161571ce8
Showing with 8 additions and 2 deletions.
  1. +8 −2 src/search.cpp
@@ -1132,8 +1132,14 @@ namespace {
{
value = -search<NonPV>(pos, ss+1, -(alpha+1), -alpha, newDepth, !cutNode);

if (doLMR && !captureOrPromotion && value > alpha)
update_continuation_histories(ss, movedPiece, to_sq(move), stat_bonus(newDepth) / 2);
if (doLMR && !captureOrPromotion)
{
int bonus = stat_bonus(newDepth) / 2;
if (value > alpha)
update_continuation_histories(ss, movedPiece, to_sq(move), bonus);
else
update_continuation_histories(ss, movedPiece, to_sq(move), -bonus);
}
}

// For PV nodes only, do a full PV search on the first move or after a fail

6 comments on commit 1b406fb

@locutus2

This comment has been minimized.

Copy link
Owner

replied Jul 11, 2019

I go now sleeping so if one of my lmr_update tests passes STC someone can start a LTC please.

@31m059

This comment has been minimized.

Copy link

replied Jul 12, 2019

@31m059

This comment has been minimized.

Copy link

replied Jul 12, 2019

Congratulations!

@mstembera

This comment has been minimized.

Copy link

replied Jul 12, 2019

Congrats!

@Vizvezdenec

This comment has been minimized.

Copy link

replied Jul 12, 2019

nice :)

@VoyagerOne

This comment has been minimized.

Copy link

replied Jul 12, 2019

@locutus2
Cool patch! Nice work!
Maybe also try updating capture stats as well.

Please sign in to comment.
You can’t perform that action at this time.