Do stats updates after LMR for captures

Vizvezdenec · vondele · commit 90d051952f4f · 2022-01-29T08:58:12.000+01:00
Since captures that are in LMR use continuation histories of corresponding quiet moves it makes sense to update this histories if this capture passes LMR by analogy to existing logic for quiet moves. Passed STC https://tests.stockfishchess.org/tests/view/61f367eef7fba9f1a4f1318b LLR: 2.94 (-2.94,2.94) <0.00,2.50> Total: 208464 W: 56006 L: 55407 D: 97051 Ptnml(0-2): 964, 23588, 54655, 23935, 1090 Passed LTC https://tests.stockfishchess.org/tests/view/61f41e34f7fba9f1a4f15241 LLR: 2.96 (-2.94,2.94) <0.50,3.00> Total: 69144 W: 18793 L: 18441 D: 31910 Ptnml(0-2): 65, 6982, 20142, 7302, 81 closes #3910 bench 4637392
diff --git a/src/search.cpp b/src/search.cpp
@@ -1194,11 +1194,14 @@ namespace {
           value = -search<NonPV>(pos, ss+1, -(alpha+1), -alpha, newDepth + doDeeperSearch, !cutNode);
 
           // If the move passed LMR update its stats
-          if (didLMR && !captureOrPromotion)
+          if (didLMR)
           {
               int bonus = value > alpha ?  stat_bonus(newDepth)
                                         : -stat_bonus(newDepth);
 
+              if (captureOrPromotion)
+                  bonus /= 4;
+
               update_continuation_histories(ss, movedPiece, to_sq(move), bonus);
           }
       }

Original file line number	Diff line number	Diff line change
`@@ -1194,11 +1194,14 @@ namespace {`
`1194`	`1194`	`value = -search<NonPV>(pos, ss+1, -(alpha+1), -alpha, newDepth + doDeeperSearch, !cutNode);`
`1195`	`1195`
`1196`	`1196`	`// If the move passed LMR update its stats`
`1197`		`- if (didLMR && !captureOrPromotion)`
	`1197`	`+ if (didLMR)`
`1198`	`1198`	`{`
`1199`	`1199`	`int bonus = value > alpha ? stat_bonus(newDepth)`
`1200`	`1200`	`: -stat_bonus(newDepth);`
`1201`	`1201`
	`1202`	`+ if (captureOrPromotion)`
	`1203`	`+ bonus /= 4;`
	`1204`	`+`
`1202`	`1205`	`update_continuation_histories(ss, movedPiece, to_sq(move), bonus);`
`1203`	`1206`	`}`
`1204`	`1207`	`}`