Skip to content

Commit

Permalink
Try the opposite: skip the OnlyPawnDefender bonus
Browse files Browse the repository at this point in the history
Bench: 3760549
  • Loading branch information
snicolet committed Sep 11, 2019
1 parent b0c07b5 commit 798a2b4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/evaluate.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -578,7 +578,7 @@ namespace {
}

// Bonus for threats on blocked pawns which has only one non-pawn defender
if (pos.rule50_count() > 6)
if (pos.rule50_count() <= 6)
{
b = pos.pieces(Them, PAWN)
& shift<Up>(pos.pieces(Us, PAWN))
Expand Down

2 comments on commit 798a2b4

@Rocky640
Copy link

Choose a reason for hiding this comment

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

Stéphane, have you tried something like replacing

& shift<Up>(pos.pieces(Us, PAWN))
with
& shift<Up>(pos.pieces(Us, PAWN) | attackedBy[Us][PAWN])

@snicolet
Copy link
Owner Author

Choose a reason for hiding this comment

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

No, I have tried shift<Up>(pos.pieces(Us, PAWN) | attackedBy2[Us]) but not your version. I will try it, thanks!

Please sign in to comment.