Skip to content

Commit

Permalink
Simplify TT penalty stat (#980)
Browse files Browse the repository at this point in the history 
STC:
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 20251 W: 3692 L: 3570 D: 12989

LTC:
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 16432 W: 2155 L: 2029 D: 12248

Bench: 5941174
  • Loading branch information
@VoyagerOne @mcostalba
VoyagerOne authored and mcostalba committed Jan 29, 2017
1 parent 5254a60 commit fa24cc2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/search.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -657,7 +657,7 @@ namespace {
update_cm_stats(ss-1, pos.piece_on(prevSq), prevSq, -stat_bonus(depth + ONE_PLY));
}
// Penalty for a quiet ttMove that fails low
else if (ttValue < alpha && !pos.capture_or_promotion(ttMove))
else if (!pos.capture_or_promotion(ttMove))
{
Value penalty = -stat_bonus(depth + ONE_PLY);
thisThread->history.update(pos.side_to_move(), ttMove, penalty);
Expand Down

2 comments on commit fa24cc2

@Stefano80
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@Kingdefender
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did think that < alpha condition was strange, it should not matter in non PV nodes, you are already <= alpha after the else. In PV nodes it's a different matter. Glad to see that you tested it and the Framework is able to see it does not matter!

Please sign in to comment.