Repository navigation
Add board insights to player stats - #143
Merged
Merged
Conversation
`player consolidate` now fills in insights read from every game's board and claims, as new PlayerStats fields (pipeline/player_insights.py): - pace: median time to their first claim, claims per hour, median and fastest winning time (linked to the game) - claim_order / claim_order_by_type: the game clock at their 1st, 2nd... 13th claim of each game, overall and per square pool, with the square claimed and everyone else's median for comparison - win_types: game wins by line, majority and points - first_square: how often they claim a game's first square, and win after it - comebacks: wins after trailing by 3+ squares, and the largest - form: last five results, longest and current streak - quickest_squares / surest_squares: squares they claim faster than everyone else, and most often when dealt A claim is a mark still held at the end of the game. In 2v2, side results count for both teammates and a claim for the teammate whose pool the square is in. New fields only, so a minor change. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
player consolidatenow adds insights read from every game's board and claims to each player'sstats.yaml, for the player page on scadustats-site (adri0/scadustats-site#76). The new code is inpipeline/player_insights.py, called afterconsolidate_playersin bothplayer consolidatepaths.New
PlayerStatsfieldspaceclaim_orderclaim_order_by_typewin_typesfirst_squarecomebacksformquickest_squaressurest_squaresDefinitions
square stats. Times are the game clock.Compatibility
New fields only, with defaults, so a minor release (1.6.0; 1.5.0 is already out without this). The site shows its Insights section only once these fields are present.
Testing
tests/pipeline/test_player_insights.py; all 570 pass, ruff clean.🤖 Generated with Claude Code