Repository navigation
feat/wsl2 - #255
feat/wsl2#255rishika-y-gaur wants to merge 8 commits into
Conversation
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Resolve the NPU metric reporting defect and prevent telemetry data loss between iterations.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 1
What changed in this PR
Adds Windows-host telemetry collection for WSL2 benchmarks and integrates GPU, power, memory, and PCM metrics into consolidated results.
Changes:
- Adds Windows telemetry collection and lifecycle management.
- Integrates collection with Docker Compose benchmark runs.
- Adds WSL2 consolidation logic and tests.
| File | Description |
|---|---|
benchmark-scripts/windows_metrics.py |
Implements Windows telemetry collection. |
benchmark-scripts/consolidate_multiple_run_of_metrics.py |
Merges Windows metrics into summaries. |
benchmark-scripts/benchmark.py |
Integrates collector lifecycle with Compose. |
benchmark-scripts/benchmark_test.py |
Tests collection and lifecycle behavior. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
🟡 Changes recommended
Unresolved telemetry defects affect PCM and GPU metrics.
1 open finding
1 resolved since last review
🧠 Review effort: Lite
Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.
There was a problem hiding this comment.
🟡 Changes recommended
PCM header handling can omit bandwidth metrics, and GPU averaging excludes idle intervals.
3 open findings
🧠 Review effort: Balanced
Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.
Signed-off-by: rishika-y-gaur <rishika.gaur@intel.com>


added windows collector for pcm, gpu, memory bandwidth
PR Checklist
What are you changing?
Adds Windows-host telemetry collection for WSL2 benchmarks and integrates GPU, power, memory, and PCM metrics into consolidated results.
Adds Windows telemetry collection and lifecycle management.
Integrates collection with Docker Compose benchmark runs.
Adds WSL2 consolidation logic and tests.
Task this PR will close
close: #340, #341, #342
Anything the reviewer should know when reviewing this PR?
Test Instructions if applicable
Test all workloads on Windows using WSL2 and Native Linux
For now WSL2 supports CPU and GPU workloads
If the there are associated PRs in other repositories, please link them here (i.e. intel-retail/performance-tools )