-
Notifications
You must be signed in to change notification settings - Fork 242
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
Farmer audit benchmark #2124
Farmer audit benchmark #2124
Conversation
219d945
to
78b3f81
Compare
d12f6aa
to
7862d93
Compare
7862d93
to
76f77bd
Compare
Windows is so annoying... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A tiny change request. Makes sense otherwise.
crates/subspace-farmer/src/bin/subspace-farmer/commands/benchmark.rs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Nazar, noted!
This adds auditing benchmark to the farmer, which can be used with existing plot. We will use this going forward to ensure changes we do to the auditing are actually improvements as well as to compare different potential implementations.
The usage is very simple:
The output will look like this:
It prints both full audit time and throughput that takes plot size into account. In example above auditing speed is ~24TiB/s.
First few commits are just refactoring, the benchmark implementation is one commit at the end.
Resolves #1914
Code contributor checklist: