Skip to content

Add run and event numbers to output tree. - #81

Merged
GernotMaier merged 7 commits into
mainfrom
gh-training
Aug 26, 2026
Merged

Add run and event numbers to output tree.#81
GernotMaier merged 7 commits into
mainfrom
gh-training

Conversation

@GernotMaier

Copy link
Copy Markdown
Member

This allows in Eventdisplay to absolutely ensure that events are aligned.

@GernotMaier GernotMaier self-assigned this Aug 26, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds run/event identifiers to XGBoost output trees for alignment verification in Eventdisplay.

Changes:

  • Writes runNumber and eventNumber to stereo and classification outputs.
  • Adds classification provenance tracking and retains size features.
  • Updates tests and changelog coverage.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/eventdisplay_ml/models.py Writes identifiers and changes classification grouping.
src/eventdisplay_ml/data_processing.py Adds provenance tracking and classification preprocessing changes.
src/eventdisplay_ml/features.py Retains classification size features.
tests/test_regression_production_guard.py Tests stereo output identifiers.
tests/test_regression_contracts.py Updates stereo writer contracts.
tests/test_models_helpers.py Tests identifier payloads.
tests/test_features.py Updates size-feature expectations.
tests/test_data_processing_remaining.py Tests provenance and size retention.
docs/changes/81.feature.md Documents output identifiers.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/eventdisplay_ml/models.py Outdated
Comment thread src/eventdisplay_ml/data_processing.py Outdated
@GernotMaier
GernotMaier marked this pull request as ready for review August 26, 2026 14:08
@GernotMaier
GernotMaier merged commit 9a6bc2b into main Aug 26, 2026
3 checks passed
@GernotMaier
GernotMaier deleted the gh-training branch August 26, 2026 18:53
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