-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Conversation
8c4eaf7
to
4b81097
Compare
4b81097
to
e699560
Compare
db3a269
to
d88b7fa
Compare
d88b7fa
to
f0c14f3
Compare
bot bench-vm -v UPSTREAM_MERGE=n $ all |
83d7d10
to
c352139
Compare
Could you give a Tldr? |
During the test runs at firsts by @bakhtin then in my tests and @ggwpez ones we didn't observe any cases of 'noise neighbors'. Also i've only heard about this but never saw real proves. More info could be found in this thread https://forum.parity.io/t/investigating-new-benchmarking-hardware/687/134 |
The only noise we still see is in 3-5 benchmarks (two of them in pallet-contracts) consistently, but that is also present on |
Here is one for BM with some fluctuations. |
What happened to paritytech/polkadot-sdk#379? |
Nothing so far. Problem is that even with very high repetitions (2000 instead of 50), some calls show consistent outlier behaviour. This lead me to de-prioritize the dynamic reps, as it wont solve that fluctuation. |
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.
After this is merged, please make another MR and then check that the bench-bot uses the new VM runners.
Needs one more approve from @paritytech/core-devs. |
@ggwpez For the benches which are noisy, maybe you could try running them with this script of mine and see if it helps? https://gist.github.com/koute/83fb86c0b80213949d2985903a03a7f8 (Make sure It applies a bunch of system-level tweaks to make benchmarking more consistent; I find that on my machine it significantly helps in CPU bound benchmarks. (Most of which we honestly should most likely permanently preapply on our benchmarking machines.) |
cb204bf
to
85b0807
Compare
f67bc8e
to
c9c84df
Compare
c9c84df
to
6bee6fc
Compare
bot bench-vm $ all |
Co-authored-by: command-bot <>
Baseline weights generated on VM runners.
https://forum.parity.io/t/investigating-new-benchmarking-hardware/687/134