Skip to content

feat/wsl2 [DO NOT MERGE] - #255

Open
rishika-y-gaur wants to merge 5 commits into
intel-retail:mainfrom
rishika-y-gaur:feat/metrics
Open

rishika-y-gaur wants to merge 5 commits into
intel-retail:mainfrom
rishika-y-gaur:feat/metrics

Conversation

@rishika-y-gaur

Copy link
Copy Markdown
Contributor

PR Checklist

  • Added label to the Pull Request for easier discoverability and search
  • Commit Message meets guidelines as indicated in the URL https://github.com/intel-retail/performance-tools/blob/main/CONTRIBUTING.md
  • Every commit is a single defect fix and does not mix feature addition or changes
  • Unit Tests have been added for new changes
  • Updated Documentation as relevant to the changes
  • All commented code has been removed
  • If you've added a dependency, you've ensured license is compatible with repository license and clearly outlined the added dependency.
  • PR change contains code related to security
  • PR introduces changes that breaks compatibility with other modules (If YES, please provide details below)

What are you changing?

Issue this PR will close

close: #issue_number

Anything the reviewer should know when reviewing this PR?

Test Instructions if applicable

If the there are associated PRs in other repositories, please link them here (i.e. intel-retail/performance-tools )

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.

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 High severity

Open (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.

if wsl2:
from windows_metrics import blank_metrics
full_kpi_dict.update(blank_metrics())
full_kpi_dict[AVG_NPU_USAGE_CONSTANT] = '0.00'
@rishika-y-gaur rishika-y-gaur changed the title Feat/metrics [DO NOT MERGE] Feat/wsl2 [DO NOT MERGE] Sep 24, 2026
@rishika-y-gaur rishika-y-gaur changed the title Feat/wsl2 [DO NOT MERGE] feat/wsl2 [DO NOT MERGE] Sep 24, 2026

This branch has not been deployed

No deployments
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