spacemeshos / gpu-post

Spacemesh proof of space time gpu optimized setup
GNU General Public License v3.0
29 stars 9 forks source link

Use current PoST settings as default for Benchmarks #91

Closed fasmat closed 1 year ago

fasmat commented 1 year ago

PoST now uses 16 byte labels and N = 8192 for initialization. This updates the benchmark to be in line with those parameters.

fasmat commented 1 year ago

Imho all tests in that file needs that adjustment. Most of them expose CLI so we need to adjust them all.

All tests have been updated with N = 8192. LabelSize is a parameter that can be passed as an argument and now defaults to 128 (=16 bytes).

Which other adjustments do you think we need?

pigmej commented 1 year ago

@fasmat I think we might need to add some info to the README.md about that change next to the benchmarks so people are not confused (having 16 less)

fasmat commented 1 year ago

Benchmarks are now tracked with MB/s instead of MH/s, since data throughput is a better measure to capture for a benchmark.

I also updated the README in 2 ways: newer benchmarks are printed higher up in the table and results are tracked with MB/s

pigmej commented 1 year ago

Apple M2 Pro: status 0, 100000 labels, 37193 labels/s, 0.57 MiB/s I like that output :)

but @fasmat we need to fix unit tests ;)

fasmat commented 1 year ago

there seems to be an issue with the arm64 build. I cannot get the one failing test to work on it 🙁