PR #563: Wire pitcher_arm_angles + bat-tracking-swing-path + refresh batter statcast data#432
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| Docker | May 15, 2026 2:23a.m. | Review ↗ | |
| JavaScript | May 15, 2026 2:23a.m. | Review ↗ | |
| Python | May 15, 2026 2:23a.m. | Review ↗ | |
| SQL | May 15, 2026 2:23a.m. | Review ↗ | |
| Secrets | May 15, 2026 2:23a.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
Up to standards ✅🟢 Issues
|
There was a problem hiding this comment.
Code Review
This pull request introduces support for pitcher arm angles and bat tracking swing path data. It updates the _load function to parse pitcher_arm_angles.csv and bat-tracking-swing-path.csv, populating a new _pitcher_arm_angles dictionary and supplementing existing batter tracking data with metrics like swing tilt and attack angle. A new helper function, get_pitcher_arm_angle, was also added to retrieve pitcher-specific arm angle information. I have no feedback to provide as there were no review comments.
PR #563 — Wire Two Unwired Statcast Files + Fresh Batter Data
Both
pitcher_arm_angles.csvandbat-tracking-swing-path.csvhave been sitting indata/statcast/since upload but were never loaded bystatcast_static_layer.py. This PR wires them in and refreshes batter statcast data.Changes
statcast_static_layer.py(+62 lines, 911 → 973)_pitcher_arm_anglesdict —{pitcher_id: {ball_angle, pitch_hand, release_z, shoulder_z, n_pitches}}pitcher_arm_angles.csv— 306 pitchers; keyed bypitcher(MLBAM ID)bat-tracking-swing-path.csv— supplements_batter_trackingwithswing_tilt,attack_angle,ideal_attack_rate,attack_directionfor ~222 battersget_pitcher_arm_angle(player_id)public functiondata/statcast/statcast_batters_2026.csvNew Signal Availability
ball_angleswing_tiltattack_angleideal_attack_rateSummary by cubic
Wired pitcher arm angles and bat swing path data into the Statcast static layer to expose new pitcher/batter signals. Also refreshed batter Statcast data.
pitcher_arm_angles.csvinto_pitcher_arm_angles.get_pitcher_arm_angle(player_id)to fetch arm slot metrics.bat-tracking-swing-path.csvto augment_batter_trackingwithswing_tilt,attack_angle,ideal_attack_rate,attack_direction.statcast_batters_2026.csvto latest.Written for commit 86dfe23. Summary will update on new commits.