Skip to content

[metrics] Added plots for metrics output - #146

Merged
AndK17 merged 9 commits into
mainfrom
metrics/new-report-formats
May 10, 2026
Merged

[metrics] Added plots for metrics output#146
AndK17 merged 9 commits into
mainfrom
metrics/new-report-formats

Conversation

@il3241ya

Copy link
Copy Markdown
Contributor

Summary

Adds automatic time-series plot generation for collected metric reports. This keeps report.json as the machine-readable output and additionally saves PNG plots next to it for quick visual inspection of metric behavior over simulation ticks.

Related issue

Closes #

Changes

Added MetricPlotBuilder for generating PNG time-series plots from existing metric report series.

  • Added centralized MetricPlotStyle / DEFAULT_METRIC_PLOT_STYLE to keep plot appearance configurable in one place.
  • Integrated plot generation into EvaluationManager, creating a plots/ directory alongside report.json.

Validation

  • Simulation run
  • Manual verification
  • Not applicable

Checklist

  • Linked the related issue
  • Kept the PR focused and reviewable
  • Updated documentation if needed
  • Added or updated validation steps if needed

@il3241ya
il3241ya requested a review from AndK17 May 10, 2026 09:16
@il3241ya il3241ya self-assigned this May 10, 2026
@il3241ya il3241ya added the enhancement New feature or request label May 10, 2026
Comment thread opencda/metrics_tools/plot_builder.py Fixed
Comment thread opencda/metrics_tools/plot_builder.py Fixed
Comment thread opencda/metrics_tools/plot_builder.py Fixed
Comment thread opencda/metrics_tools/plot_builder.py Fixed
Comment thread opencda/metrics_tools/plot_builder.py Dismissed
@il3241ya
il3241ya marked this pull request as ready for review May 10, 2026 10:36
@AndK17
AndK17 merged commit 7264b1d into main May 10, 2026
7 checks passed
@AndK17
AndK17 deleted the metrics/new-report-formats branch May 10, 2026 17:44
meaning-0f-life pushed a commit to meaning-0f-life/OpenCDA that referenced this pull request Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants