Skip to content

[codex] Add analysis graph hooks for ndarray features#29

Draft
colonelpanic8 wants to merge 3 commits into
masterfrom
codex/ndarray-analysis-features
Draft

[codex] Add analysis graph hooks for ndarray features#29
colonelpanic8 wants to merge 3 commits into
masterfrom
codex/ndarray-analysis-features

Conversation

@colonelpanic8
Copy link
Copy Markdown
Contributor

Summary

  • Adds optional analysis graph hooks to ndarray global and per-player feature adders.
  • Lets NDArrayCollector build and evaluate one shared analysis graph from the union of feature-adder dependencies.
  • Reuses analysis frame input bookkeeping between the analysis collector and ndarray runtime.
  • Adds a regression test proving two ndarray adders share one graph evaluation per emitted row.

Validation

  • cargo test analysis_feature_adders_share_one_graph_evaluation_per_ndarray_row
  • cargo test --lib
  • cargo test currently hits a pre-existing tests/stats_frame_resolution_test.rs failure: events.boost_pickups.len: left=0, right=557; reproduced on clean origin/master.

@colonelpanic8 colonelpanic8 force-pushed the codex/ndarray-analysis-features branch from 1fdde78 to 880672c Compare May 22, 2026 03:14
@colonelpanic8 colonelpanic8 force-pushed the codex/ndarray-analysis-features branch from 880672c to 02fbe9d Compare May 22, 2026 03:23
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