Skip to content

Complete measured runtime overhead reductions#36

Open
PrimeBuild-pc wants to merge 6 commits into
perf/minimize-runtime-overheadfrom
perf/minimize-runtime-overhead-next
Open

Complete measured runtime overhead reductions#36
PrimeBuild-pc wants to merge 6 commits into
perf/minimize-runtime-overheadfrom
perf/minimize-runtime-overhead-next

Conversation

@PrimeBuild-pc

@PrimeBuild-pc PrimeBuild-pc commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Summary

Follow-up to #35, containing the measured P1 and justified P2 work from the performance plan:

  • defer inactive Power Plans, Rules, Settings, Tweaks, and Logs page initialization
  • reuse the process monitor's startup snapshot
  • parse the active plan from powercfg /list, removing redundant powercfg calls
  • run the power-plan timer only while the Power Plans page is visible
  • suspend hidden Log Viewer collection updates and arm log flushing only when entries are queued
  • back off failed WMI recovery attempts (10s, 30s, 60s, 5m)
  • reuse static process paths with PID + start-time validation

This PR is intentionally stacked on #35. It does not change version metadata or issue #32 priority verification/retry behavior.

Final measurements vs official v1.4.3

Scenario Avg CPU Working set Handles max Child processes
visible idle 27.6748% → 28.8136% 301.12 → 299.25 MB 1767 → 1704 27 → 4
hidden idle 0.0878% → 0.0000% 146.10 → 140.24 MB 705 → 622 0 → 0
hidden + 80 process churn 0.2107% → 0.1931% 148.69 → 141.93 MB 707 → 617 0 → 0

The visible CPU delta is within run noise, while foreground child process launches fell from 27 to 4. Hidden CPU reached zero in the idle sample, memory fell 4–5%, and handle maxima fell materially.

Validation

  • clean, restore, Release build: 0 warnings/errors
  • 581/581 tests passed
  • official WinX64-SingleFile publish succeeded
  • NuGet audit: no vulnerable packages
  • persistent rules, process start/stop, lazy diagnostics, and issue Can't save rule #32 retry tests remain green

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@codecov-commenter

codecov-commenter commented Jul 11, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 37.96296% with 67 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
MainWindow.Behaviors.partial.cs 0.00% 23 Missing ⚠️
Services/ProcessService.cs 44.82% 15 Missing and 1 partial ⚠️
Services/ProcessMonitorService.cs 34.78% 14 Missing and 1 partial ⚠️
Services/EnhancedLoggingService.cs 16.66% 5 Missing ⚠️
ViewModels/ProcessPowerPlanAssociationViewModel.cs 0.00% 4 Missing and 1 partial ⚠️
ViewModels/LogViewerViewModel.cs 60.00% 2 Missing ⚠️
MainWindow.xaml.cs 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@PrimeBuild-pc PrimeBuild-pc force-pushed the perf/minimize-runtime-overhead-next branch from be4d9af to ed863cd Compare July 11, 2026 23:24
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