Skip to content
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

results in README #4

Closed
rchikhi opened this issue May 6, 2019 · 2 comments
Closed

results in README #4

rchikhi opened this issue May 6, 2019 · 2 comments

Comments

@rchikhi
Copy link
Collaborator

rchikhi commented May 6, 2019

add a table in the README with results (in MB/s) for e.g. counting lines and outputting whole text, versus gunzip

as results from the paper will likely be improved

@Piezoid
Copy link
Owner

Piezoid commented May 6, 2019

Good idea, indeed.

There is few tweaks I want to try first.

I can benchmark on my 2011's 12 cores machine. Tell me if you have another idea.

There is also the CRC32 issue, since we have it disabled at the moment it wouldn't be completely fair to compare ourself to other implementations that compute it.

@rchikhi
Copy link
Collaborator Author

rchikhi commented May 6, 2019

Good points. Any machine will do, really. And for CRC32, we'll just make a note (asterisk) to indicate this caveat (until CRC32 is implemented but i don't see it as high priority).

@rchikhi rchikhi closed this as completed May 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants