Repository navigation
Eval: Selective pawn-backed knight outposts (replace losing heuristics) - #46
Open
KnightStackSys wants to merge 6 commits into
Open
KnightStackSys wants to merge 6 commits into
KnightStackSys wants to merge 6 commits into
Conversation
KnightStackSys
marked this pull request as ready for review
October 20, 2026 14:50
This branch has not been deployed
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.
Why rework this branch
The full knight check/development/proximity experiment reportedly lost approximately 20 Elo in a completed strength test. Rather than tuning every speculative positional reward, this revision disables all four failed heuristics by default and replaces them with a conservative, testable pawn-backed knight outpost.
Revised default
EvalKnightCheckThreatPenalty,EvalKnightDevelopmentBonus,EvalKnightEnemyPressureBonus, andEvalKnightKingApproachBonusknobs now all default to 0; they remain available to isolate effects in experiments.EvalKnightOutpostBonusoption defaults to 6, range 0–50; 0 disables the new outpost as well. All new HCE evaluations short-circuit when disabled.ironphoenix_knight_outpostcovers king/partner score signs, pawn support, enemy pawn challenge, all four player orientations, outpost team cap, eliminated pieces, enemy presence, center neutrality, and UCI tuning.Important
This is an untested-for-Elo revised hypothesis, not a verified fix for the −20 Elo regression. The team-centric pawn-defended flank criteria are stricter, but the heuristic can still lose games or change search speed. Benchmarks and matched completed SPRT remain required before merging. NNUE is unaffected. No changes to
main.Validation
Linux and Windows builds / CTest running on this commit. See
docs/knight-activity.mdfor exact criteria and A/B settings.