[3/6] Add MVV-LVA capture ordering and killer moves heuristic#35
Open
luccabb wants to merge 6 commits intofeature/transposition-table-boundsfrom
Open
[3/6] Add MVV-LVA capture ordering and killer moves heuristic#35luccabb wants to merge 6 commits intofeature/transposition-table-boundsfrom
luccabb wants to merge 6 commits intofeature/transposition-table-boundsfrom
Conversation
e8b9ff6 to
df8a32b
Compare
0b53062 to
944a0e0
Compare
df8a32b to
bfb11c1
Compare
944a0e0 to
f1001bf
Compare
4f7fad5 to
1c7b8d9
Compare
f1001bf to
98ee9e1
Compare
🔬 Stockfish Benchmark Resultsvs Stockfish Skill Level 3
Non-checkmate endings:
vs Stockfish Skill Level 4
Non-checkmate endings:
vs Stockfish Skill Level 5
Non-checkmate endings:
Configuration
|
🔬 Stockfish Benchmark Resultsvs Stockfish Skill Level 3
Non-checkmate endings:
vs Stockfish Skill Level 4
Non-checkmate endings:
vs Stockfish Skill Level 5
Non-checkmate endings:
Configuration
|
Improves move ordering for better pruning: **MVV-LVA (Most Valuable Victim - Least Valuable Attacker):** - Sort captures by MVV-LVA score in `organize_moves()` - Best captures (e.g., pawn takes queen) searched first - Increases beta cutoff rate for better pruning **Killer Moves Heuristic:** - Track quiet moves that cause beta cutoffs (2 per ply) - Add `killers` parameter to `organize_moves()` and `negamax()` - Killer moves searched after captures, before other quiet moves - Killers stored in a table indexed by ply depth **Move Ordering Priority:** 1. Captures (sorted by MVV-LVA) 2. Killer moves (quiet moves that caused cutoffs at this ply) 3. Other quiet moves (shuffled for variety) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
8bab1bf to
e73ebd6
Compare
fbfac44 to
6912b81
Compare
Owner
Author
|
/run-nps-benchmark |
⚡ NPS Benchmark Results
Per-position breakdown |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Details
MVV-LVA Capture Ordering
Captures are now sorted by MVV-LVA score, putting the best captures first:
Killer Moves Heuristic
Quiet moves that caused beta cutoffs are remembered:
Move Ordering Priority
Test plan
🤖 Generated with Claude Code