Skip to content

Fix Lite perfmon chart bugs and Dashboard ScottPlot crash handling#546

Merged
erikdarlingdata merged 1 commit intodevfrom
fix/issue-544-545-perfmon-and-scottplot
Mar 13, 2026
Merged

Fix Lite perfmon chart bugs and Dashboard ScottPlot crash handling#546
erikdarlingdata merged 1 commit intodevfrom
fix/issue-544-545-perfmon-and-scottplot

Conversation

@erikdarlingdata
Copy link
Owner

Summary

Changes

  • Lite chart query: SUM(delta_cntr_value) GROUP BY collection_time so multi-instance counters (e.g. "Query optimizations/sec" with "default"/"internal") render as a single clean line
  • DeltaCalculator: Added timestamp tracking and gap detection — if >5min since last cached value, returns 0 delta instead of an inflated value spanning the entire gap. All seed methods now store timestamps from DuckDB.
  • Perfmon collector: Passes collectionTime and maxGapSeconds: 300 to delta calculator. Fixed sample_interval_seconds from hardcoded 600 to 60.
  • Dashboard: Wrapped ServerTab construction in try-catch with error message pointing to VC++ Redistributable download

Test plan

  • Launch Lite, connect to a server, let it collect for a few minutes
  • Check Perfmon tab — counters like "Query optimizations/sec" and "Network IO waits" should show single smooth lines
  • Stop and restart Lite — first data point after restart should not spike
  • Dashboard opens server tabs normally (no regression from try-catch)

Closes #544, closes #545

🤖 Generated with Claude Code

, #545)

Lite perfmon fixes (#544):
- Aggregate chart query by collection_time (SUM across instance_name)
  so multi-instance counters show one clean line instead of jagged duplicates
- Add gap detection to DeltaCalculator — if >5min since last cached value,
  return 0 delta instead of inflated value spanning the entire gap
- Seed timestamps from DuckDB so gap detection works after app restarts
- Fix sample_interval_seconds from hardcoded 600 to 60 (actual interval)

Dashboard fix (#545):
- Wrap ServerTab construction in try-catch with user-friendly error message
  pointing to VC++ Redistributable download for SkiaSharp failures

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit c0b1dec into dev Mar 13, 2026
3 checks passed
@erikdarlingdata erikdarlingdata deleted the fix/issue-544-545-perfmon-and-scottplot branch March 13, 2026 17:25
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.

1 participant