Cyan4973 / Cyan4973/xxHash

Include CRC32 in benchmarks on website

Open
#1,050 1 comment 2 reactions 0 assignees View on GitHub
Dominant language
C
Stars
11.2k
Forks
913
Avg merge
2h 58m
Merged PRs (30d)
24

Description

Looking at benchmarks table on main website I noticed lack off most popular check sum on internet, namely CRC32. Although it's not a hash but checksum but 'bandwidth speed' in this table refers to hashes being used add check sums, hence proposition to include CRC.

[Here](https://paste.ee/p/qUxbgCIc) is improvised patch with it.

Also, suggetion to a bench - every test should anounce units it uses, as these numbers don't mean much, if anything, without it.

~~~
benchmarking large inputs : from 512 bytes (log9) to 128 MB (log27) [MB/s]:
Throughput small inputs of fixed size (from 1 to 30 bytes) [#/s]:
benchmarking random size inputs [1-N] ???:
~~~

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the benchmark table on the main website and the improvised patch linked in the issue; the payload does not identify source files or tests. Done means CRC32 is included in the displayed benchmarks and the listed measurements clearly state their units.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
documentation, performance
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.