Page MenuHomePhabricator
Differential D462

Benchmark_report class
Needs ReviewPublic
Actions

Authored by tfu on Sep 15 2021, 1:52 AM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Maniphest Tasks
T1505: Benchmark_Report class
Summary

Renamed class and added function

Test Plan

TDB

Diff Detail

Repository
rG Goldilocks
Branch
Benchmark_report
Lint
Lint Passed

Event Timeline

tfu requested review of this revision.Sep 15 2021, 1:52 AM
tfu created this revision.
Restricted Application completed remote builds in Restricted Buildable.Sep 15 2021, 1:52 AM
tfu updated this revision to Diff 2046.Sep 15 2021, 10:03 PM

Updated print function

Restricted Application completed remote builds in Restricted Buildable.Sep 15 2021, 10:03 PM
tfu planned changes to this revision.Sep 15 2021, 10:04 PM

I was not able to finish this class. It will need further work to be considered complete.

tfu retitled this revision from Bencmark_report class to Benchmark_report class.Sep 15 2021, 10:05 PM
tfu abandoned this revision.Sep 28 2021, 4:09 PM
tfu updated this revision to Diff 2086.Oct 23 2021, 2:21 AM
  • Found bug in code
Restricted Application completed remote builds in Restricted Buildable.Oct 23 2021, 2:21 AM
tfu added a comment.Oct 23 2021, 2:23 AM

I found an unexpected bug in the code for the function finalize. I will now have to make edits to the code which will take an extra week to two weeks.

tfu added a task: T1505: Benchmark_Report class.Oct 23 2021, 2:23 AM
tfu mentioned this in T1511: Results vector modification needed.Oct 23 2021, 2:29 AM
tfu updated this revision to Diff 2095.Nov 11 2021, 1:49 AM
  • Changes planned
  • Fixing bug attempt 1
Restricted Application completed remote builds in Restricted Buildable.Nov 11 2021, 1:49 AM
tfu added a comment.Nov 11 2021, 1:51 AM

Need to make changes to result, related to benchmark_runner.

tfu updated this revision to Diff 2096.Nov 18 2021, 9:48 PM
  • Added calculate function
  • Merge branch 'devel' into Benchmark_report
  • Added test case in main.cpp
  • fixed cMake error
  • Detail fix
  • type mismatch fix
  • Fixed merge issues
  • Test type mismatch fix
  • Continuing fixing bug
Restricted Application completed remote builds in Restricted Buildable.Nov 18 2021, 9:48 PM
tfu updated this revision to Diff 2103.Dec 10 2021, 6:58 PM

Updated code to fix bug

Restricted Application completed remote builds in Restricted Buildable.Dec 10 2021, 6:58 PM

Revision Contents
Changeset List

PathSize
3 lines
4 lines
goldilocks-source/
5 lines
include/
goldilocks/
benchmark_report.hpp
benchmark_results.hpp
119 lines
12 lines
18 lines
src/
benchmark_report.cpp
benchmark_results.cpp
39 lines
goldilocks-tester/
22 lines

Diff 2103

View Options

.arcconfig

Loading...
View Options

.arclint

Loading...
View Options

goldilocks-source/CMakeLists.txt

Loading...
View Options

goldilocks-source/include/goldilocks/benchmark_report.hpp

Loading...
View Options

goldilocks-source/include/goldilocks/benchmark_results.hpp

Loading...
View Options

goldilocks-source/include/goldilocks/runner.hpp

Loading...
View Options

goldilocks-source/include/goldilocks/test.hpp

Loading...
View Options

goldilocks-source/src/benchmark_report.cpp

Loading...
View Options

goldilocks-source/src/benchmark_results.cpp

Loading...
View Options

goldilocks-tester/main.cpp

Loading...
Copyright (C) 2019 MousePaw Media. All Rights Reserved. · Community Rules · Terms of Use · Privacy Policy · Terms of Development