Plotting script added#5
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a standalone plotting utility to generate bar-chart comparisons from aggregated experiment CSV output, and updates repo config to support the new workflow.
Changes:
- Added
plot_results.pyCLI to loadall_results.csv, filter scenario/algorithm, and write per-metric PNG plots. - Added
pandasandmatplotlibdependencies. - Updated
.gitignoreto ignore generatedplots/output and log files.
Reviewed changes
Copilot reviewed 1 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
requirements.txt |
Adds plotting-related Python dependencies (pandas, matplotlib). |
plot_results.py |
New plotting script that reads aggregated CSVs and generates per-metric bar charts. |
.gitignore |
Ignores generated plot artifacts and *.log files. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| return max(candidates, key=lambda p: p.stat().st_mtime) | ||
|
|
||
|
|
||
| def validate_columns(df: pd.DataFrame, metrics: list[str]) -> None: |
There was a problem hiding this comment.
metrics: list[str] uses PEP 585 built-in generics, which require Python 3.9+. The README states the project supports Python 3.8+, so this annotation will raise a TypeError on 3.8 at import time. Consider switching to typing.List[str]/Sequence[str], or adding from __future__ import annotations, or updating the documented minimum Python version.
…ation according to number of packets fails.
No description provided.