Skip to content

Add Astros MLB scoreability fixture and regression checks#1

Merged
JakenHerman merged 6 commits intomasterfrom
cursor/astros-scoreability-tests-a433
Apr 9, 2026
Merged

Add Astros MLB scoreability fixture and regression checks#1
JakenHerman merged 6 commits intomasterfrom
cursor/astros-scoreability-tests-a433

Conversation

@JakenHerman
Copy link
Copy Markdown
Owner

@JakenHerman JakenHerman commented Apr 9, 2026

Summary

  • add a reproducible Astros scoreability pull script (scripts/pull_astros_scoreability.py)
  • add alias-aware event mapping so scoreable MLB events (e.g. field_out) are recognized
  • support targeting a specific game via --game-pk
  • add grouped issue-draft generation by unsupported event type (instead of one issue per play)
  • add README documentation for how to run the pull script and where outputs are written
  • treat script outputs as generated artifacts:
    • ignore /test_data/mlb/ in git
    • untrack previously committed JSON and issue-draft outputs

Requested game result (Astros vs Colorado)

  • Target game: gamePk=824374 (official date 2026-04-08)
  • Total plays: 82
  • Scoreable by current full-count model: 81
  • Unscoreable: 1 (sac_bunt)

Notes

  • Generated fixtures and issue drafts still exist when the script is run locally, but they are no longer committed to the repository.
  • Test suite remains green after removing fixture-dependent checks.
Open in Web Open in Cursor 

cursoragent and others added 6 commits April 9, 2026 02:40
Co-authored-by: Jaken Herman <JakenHerman7@Gmail.com>
Co-authored-by: Jaken Herman <JakenHerman7@Gmail.com>
Co-authored-by: Jaken Herman <JakenHerman7@Gmail.com>
Co-authored-by: Jaken Herman <JakenHerman7@Gmail.com>
Co-authored-by: Jaken Herman <JakenHerman7@Gmail.com>
Co-authored-by: Jaken Herman <JakenHerman7@Gmail.com>
@JakenHerman JakenHerman marked this pull request as ready for review April 9, 2026 18:11
@JakenHerman JakenHerman merged commit e86c2fd into master Apr 9, 2026
2 checks passed
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.

2 participants