-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add support for automated benchmarking #54
base: master
Are you sure you want to change the base?
Conversation
Ping. |
Sorry for the late reply. I started looking into this but then I got distracted by other urgent matters. But let's get to the point. I have doubts about this being the most proper way to implement automated benchmarking, for various reasons.
Given the last point above, I'd rather postpone the implementation of this feature until the current reporting system is replaced with something more reliable. |
On Mon, 27 Jan 2014 02:52:08 -0800
How so? I run with 6 cores (= 6 threads), and it gives reliable results |
In my experience, the results are only acceptable under certain conditions. For instance, Intel CPUs that support hyper-threading can give particularly unreliable results. |
On Mon, 27 Jan 2014 03:51:10 -0800
Wouldn't the averaging help that? The average of, say, 10 rounds would |
To some extent, but as I said I don't think that's the most proper way to do it. I would rather have just one round and make sure that the thread hash rates refer more or less to the same time span, and above all, that none is missing. |
This allows easy integration into benchmarking scripts; previously you'd need to kill it with a signal and then try to find the line.
The endless benchmarking mode is still supported, but now you can optionally specify the amount of rounds, and the total result is averaged over all these rounds.