Support custom summarize timestamp column#40
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 792112fdff
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - `python -m telemetry_window_demo.cli summarize --input data/raw/sample_events.jsonl` | ||
| - `python -m telemetry_window_demo.cli summarize --input events.csv --timestamp-col event_time` | ||
|
|
||
| That command reads `data/raw/sample_events.jsonl` and regenerates: |
There was a problem hiding this comment.
Point output artifact claim to the run command
After adding the summarize examples, the sentence “That command reads data/raw/sample_events.jsonl and regenerates:” now follows the summarize bullets, but summarize only prints stats and does not write features.csv, alerts.csv, or plots. This README flow now misstates actual CLI behavior and can mislead users reproducing outputs; the sentence should explicitly reference python -m telemetry_window_demo.cli run --config configs/default.yaml (or be moved back under that command).
Useful? React with 👍 / 👎.
Summary
Tests