Skip to content
This repository has been archived by the owner on Jan 3, 2022. It is now read-only.

Sort by Fresh Build Time #125

Merged
merged 1 commit into from Apr 8, 2020
Merged

Sort by Fresh Build Time #125

merged 1 commit into from Apr 8, 2020

Conversation

ghost
Copy link

@ghost ghost commented Apr 8, 2020

Hi,

This sorts the Benchmarks by Fresh Build Time. I felt the Fresh Built Time column has, overall, a more honest grasp of the expected performance of a machine in a development environment, so I picked it over the Incremental Build Time.

I draw my opinion from having worked with these machines:

  • 13" 2017,
  • 15" 2009,
  • 15" 2015,
  • 15" 2016,
  • Mini 2014,
  • Mac Pro 2013

Looking at the incremental build times in these benchmarks, I believe it may give a false sense of a not so significant performance improvement between the machines.

Note:

I used Markdown Table Formatter for Visual Studio Code to do this.

#36

Cheers

Copy link
Owner

@ashfurrow ashfurrow left a comment

Choose a reason for hiding this comment

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

That's cool, thanks for the PR and the explanation of your process 👍 I agree that fresh build time is probably a better indicator of performance, and it has a wider spread so the sorting will be more meaningful. Thanks again!

@ashfurrow ashfurrow merged commit e11b0fd into ashfurrow:master Apr 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant