Skip to content

Season 3 playoffs are scored on points too - #144

Merged
adri0 merged 1 commit into
mainfrom
season-3-playoffs-points
Oct 24, 2026
Merged

adri0 merged 1 commit into
mainfrom
season-3-playoffs-points

Conversation

@adri0

@adri0 adri0 commented Oct 24, 2026

Copy link
Copy Markdown
Owner

Season 3's playoffs (the semi-final and the final, double_elimination) are scored on points like its round robin: first to 13 points, a point per square plus 2 per completed line. In the semi-final VOD both games go on after a line and end when a side reaches 13.

This commit (a179804) was meant for #139, but it was pushed to season-3 after #139 merged, so it never reached main. With the released 1.5.1, the site's Season 3 playoff matches fail season_scoring ("a season 3 double_elimination game is lockout"): site PRs #157 and #158.

  • SeasonFormat.round_robin_scoring → scoring: one scoring for every game of the season.
  • validation.scoring_for(season) no longer takes the match type. Extraction and the season_scoring rule use it.
  • Docs (CLAUDE.md, match validate help) and tests updated.

Cherry-picked onto main. One conflict in extract.py with the time-limit game_end loop (#140), where I kept main's loop and switched it to the new call.

Checks: 576 tests pass, Ruff clean. scadustats match validate from this branch on site branch season-3-page (every Season 3 match, playoffs included): 0 of 134 matches have issues.

After merge: release, so the site's CI (scadustats@latest) passes on #157 and #158.

🤖 Generated with Claude Code

The semi-final VOD shows both games going on after a line, each ending
when a side reached 13 points. SeasonFormat.round_robin_scoring becomes
scoring (every game of the season), and scoring_for no longer takes the
match type.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@adri0
adri0 merged commit 8f215c1 into main Oct 24, 2026
1 check passed
@adri0
adri0 deleted the season-3-playoffs-points branch October 24, 2026 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant