Skip to content
This repository has been archived by the owner on Sep 23, 2021. It is now read-only.

Updates Local Benchmark Results #26

Merged
merged 1 commit into from
Aug 30, 2018

Conversation

juanpedromoreno
Copy link
Member

This PR updates the benchmark results for the local environment.

Note 1: I had to update the command line instruction to run the benchmarks locally , in order run them faster and consistently with the previous version.

r and w are specifying the minimum time (seconds) to spend at each measurement warmup iteration/iteration.

Note 2: GCP result will be updated in a separated PR.

Fixes #25

Copy link

@L-Lavigne L-Lavigne left a comment

Choose a reason for hiding this comment

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

Good stuff, thanks JP!

I also see that your graphs are correcting the typo from #25. (EDIT: ah you mentioned that already) :-)

I think there's still other graphs to fix before closing that one, we can do that as part of the GCP result run.

@juanpedromoreno
Copy link
Member Author

Yes, that would be the idea ;)

Thanks for the review.

@juanpedromoreno juanpedromoreno merged commit 56898c9 into UpgradeToNewHotness Aug 30, 2018
@juanpedromoreno juanpedromoreno deleted the jp-update-local-benchmarks branch August 30, 2018 20:48
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.

2 participants