Skip to content

A few fixes #15

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

A few fixes #15

wants to merge 7 commits into from

Conversation

Code0x58
Copy link

@Code0x58 Code0x58 commented Oct 15, 2017

Here are a few fixes that I needed to get the code running smoothly locally:

  • disable caching while building
  • update uvloop version so there isn't an exception caused by a missing module/member in the containers
  • hammer away at the docker container to make sure it has stopped before another one tries to start
  • make sure only one type of benchmark is running at a time so the HTML formatting doesn't blow up, seemed reasonable (and simpler) to add a constraint
  • nice to have updated kbKiB as kb wasn't kilobits
  • useful updated README.md to give versions for the required software that are known to work (including wrk which wasn't listed), and added a working example of run_benchmarks

@Code0x58 Code0x58 force-pushed the master branch 3 times, most recently from c32589a to ab84058 Compare October 16, 2017 19:58
@Code0x58
Copy link
Author

poke is this satisfactory now?

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