Missing Attributes

[colonelpanic8]

• pad availability/boost pickup
• goal-scored /attribution in frames data
• demolish attribution
• who got last touch

[colonelpanic8]

I think this is effectively done now.

Here’s where each checkbox landed:

• pad availability/boost pickup

  • Detection/extraction landed in fae0603 (feat: add ballchasing comparison tooling), which added boost_pad_events, BoostPadEventKind::{PickedUp, Available}, and the per-frame/current-frame plumbing in ReplayProcessor.
  • It was then surfaced on ReplayData in 4fbdb73 (expose exact replay events in replay data).

• goal-scored / attribution in frames data

  • Exact goal event extraction also landed in fae0603, including goal_events and current_frame_goal_events.
  • Scorer attribution was improved in fe5d231 (use goal explosion actors as scorer hints).
  • The exact goal events were exposed on ReplayData in 4fbdb73.
  • One nuance: this is exposed today as frame-keyed top-level goal_events, not as extra fields embedded directly inside frame_data.metadata_frames.

• demolish attribution

  • The original support landed in 0e16138 (Add demolish properties to replay processor).
  • It was later hardened for new-format replays in PR #25 / 2220650 (Fix demolish mapping for new-format replay).

• who got last touch

  • Touch event extraction landed in fae0603 with touch_events and current_frame_touch_events.
  • The attribution heuristic was improved in bced2aa (improve touch attribution heuristic).
  • Attribution confidence was added in 7bca5f6 (record touch attribution confidence).
  • The exact touch events were exposed on ReplayData in 4fbdb73.

There’s also test coverage now for the main pieces:

• exact goal extraction/scorer coverage
• touch attribution quality against resolved goal scorers
• new-format demolition regression coverage

Given that, I think the issue is closeable unless we specifically want to keep it open until goal attribution is embedded directly into frame_data rather than exposed as frame-indexed event streams.