Skip to content

Dev benchmarks - #8

Open
ggfagiuoli wants to merge 302 commits into
cnr-isti-vclab:mainfrom
ggfagiuoli:dev_benchmarks
Open

Dev benchmarks#8
ggfagiuoli wants to merge 302 commits into
cnr-isti-vclab:mainfrom
ggfagiuoli:dev_benchmarks

Conversation

@ggfagiuoli

@ggfagiuoli ggfagiuoli commented Mar 26, 2025

Copy link
Copy Markdown
Contributor
  • Added BenchmarkViewerDrawer, various Automations, Printers and Metrics to allow for benchmarking
  • Added Example 983 to showcase the BenchmarkViewerDrawer
  • Changed Timer so that it may be restarted after being stopped
  • Changed DesktopTrackball so that you may ask it to ignore events

ggfagiuoli and others added 5 commits March 28, 2025 16:59
…e possibility of Metrics that measure more than once per automation. BenchmarkPrinters have been modified as well to accomodate for this
@alemuntoni

alemuntoni commented Mar 31, 2025

Copy link
Copy Markdown
Member

TODO:

major:

  • can the benchmark drawer be moved and become part of the AbstractViewerDrawer? These two entities are intrinsically bound, and keep them split seems like a forced design choice (see the usage of TrackballEventIgnoreAutomationAction). We should discuss this - @malomo opinion needed!

minor/refactor:

  • remove all the printf calls - use std::cout and std::cerr instead
  • use doxygen syntax for documentation - see other classes in the library for consistency

…ream functions, unified startIgnoringTrackBallEvents() and stopIgnoringTrackBallEvents() to a single function that takes a bool parameter
…they use doxygen syntax; Removed another useless override
…w throws an actual exception, fixed BenchmarkDrawer documentation
…rackballEventIgnoreAutomationAction to TrackBallEventIgnoreAutomationAction for consistency
…e than necessary and an extra frame).

Added MultipleFpsBenchmarkMetric and MultipleTimeBenchmarkMetric, which take a measurement each frame and then calculate n averages (or sums) on the measurements
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.

3 participants