Skip to content

DA-102: Desktop app benchmarking tool#7

Closed
danmiller973 wants to merge 4 commits into6-upgrade-to-net-6from
DA-102
Closed

DA-102: Desktop app benchmarking tool#7
danmiller973 wants to merge 4 commits into6-upgrade-to-net-6from
DA-102

Conversation

@danmiller973
Copy link
Copy Markdown
Contributor

@tylerkron Does this look about right?

  • Added a new flyout for the summary view.
  • Added a SummaryLogger to calculate some metrics based on the last N DataSample's that come from the LoggingManager

danmiller973 and others added 4 commits August 11, 2022 16:28
# Conflicts:
#	Daqifi.Desktop.Bootloader.Test/Daqifi.Desktop.Bootloader.Test.csproj
#	Daqifi.Desktop.Common.Test/Daqifi.Desktop.Common.Test.csproj
#	Daqifi.Desktop.DataModel.Test/Daqifi.Desktop.DataModel.Test.csproj
#	Daqifi.Desktop.IO.Test/Daqifi.Desktop.IO.Test.csproj
#	Daqifi.Desktop.Test/Daqifi.Desktop.Test.csproj
Copy link
Copy Markdown
Contributor

@tylerkron tylerkron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You'll want to change the branch here, but then merge it!

@danmiller973 danmiller973 deleted the DA-102 branch August 26, 2022 15:44
tylerkron added a commit that referenced this pull request Apr 5, 2026
- Add suppressInitialWarning parameter to StartMonitoring so the
  pre-session warning and the first timer tick don't both show a
  warning dialog (qodo #6)
- Add lock around timer state and _warningRaised to prevent races
  between the timer callback and UI-thread start/stop calls (qodo #7)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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