Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Baseline weights VM runners #13933

Merged
merged 1 commit into from
Jun 19, 2023
Merged

Conversation

mateo-moon
Copy link
Contributor

@mateo-moon mateo-moon added the A3-in_progress Pull request is in progress. No review needed at this stage. label Apr 17, 2023
@mateo-moon mateo-moon self-assigned this Apr 17, 2023
@mateo-moon mateo-moon force-pushed the oleg/ci-cd/vm_weights_baseline branch from 8c4eaf7 to 4b81097 Compare April 18, 2023 11:44
@mateo-moon mateo-moon changed the title mock commit Baseline weights VM runners Apr 18, 2023
@mateo-moon mateo-moon closed this May 1, 2023
@mateo-moon mateo-moon force-pushed the oleg/ci-cd/vm_weights_baseline branch from 4b81097 to e699560 Compare May 1, 2023 10:38
@paritytech paritytech deleted a comment from command-bot bot May 1, 2023
@paritytech paritytech deleted a comment from command-bot bot May 1, 2023
@mateo-moon mateo-moon reopened this May 1, 2023
@mateo-moon mateo-moon force-pushed the oleg/ci-cd/vm_weights_baseline branch from db3a269 to d88b7fa Compare May 2, 2023 09:57
@mateo-moon mateo-moon closed this May 9, 2023
@mateo-moon mateo-moon force-pushed the oleg/ci-cd/vm_weights_baseline branch from d88b7fa to f0c14f3 Compare May 9, 2023 11:00
@paritytech paritytech deleted a comment from paritytech-processbot bot May 9, 2023
@paritytech paritytech deleted a comment from command-bot bot May 9, 2023
@paritytech paritytech deleted a comment from command-bot bot May 9, 2023
@ggwpez
Copy link
Member

ggwpez commented May 11, 2023

@ggwpez This is what i've told you about) I will run them separately.

Dont tell me, tell @mordamax 😆

@mateo-moon
Copy link
Contributor Author

bot bench-vm -v UPSTREAM_MERGE=n $ all

@mateo-moon mateo-moon reopened this May 15, 2023
@mateo-moon mateo-moon force-pushed the oleg/ci-cd/vm_weights_baseline branch from 83d7d10 to c352139 Compare May 16, 2023 10:35
@paritytech paritytech deleted a comment from command-bot bot May 16, 2023
@paritytech paritytech deleted a comment from paritytech-cicd-pr May 16, 2023
@paritytech paritytech deleted a comment from command-bot bot May 16, 2023
@paritytech paritytech deleted a comment from command-bot bot May 16, 2023
@paritytech paritytech deleted a comment from command-bot bot May 16, 2023
@paritytech paritytech deleted a comment from command-bot bot May 16, 2023
@paritytech paritytech deleted a comment from command-bot bot May 16, 2023
@paritytech paritytech deleted a comment from command-bot bot May 16, 2023
@paritytech paritytech deleted a comment from command-bot bot May 16, 2023
@paritytech paritytech deleted a comment from command-bot bot May 16, 2023
@paritytech paritytech deleted a comment from command-bot bot May 16, 2023
@bkchr
Copy link
Member

bkchr commented May 18, 2023

Did we already solve the benchmarking noise on these "machines"?

We didn't, because there wasn't one in the first place)

Could you give a Tldr?

@mateo-moon
Copy link
Contributor Author

mateo-moon commented May 18, 2023

Did we already solve the benchmarking noise on these "machines"?

We didn't, because there wasn't one in the first place)

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

@ggwpez
Copy link
Member

ggwpez commented May 19, 2023

Did we already solve the benchmarking noise on these "machines"?

The only noise we still see is in 3-5 benchmarks (two of them in pallet-contracts) consistently, but that is also present on bm3. So I assume that these benches are unstable themselves, and not a hardware problem.
@oleg-plakida can you maybe post some SWC links from the past VM+BM runs to demonstrate?

@mateo-moon
Copy link
Contributor Author

Did we already solve the benchmarking noise on these "machines"?

The only noise we still see is in 3-5 benchmarks (two of them in pallet-contracts) consistently, but that is also present on bm3. So I assume that these benches are unstable themselves, and not a hardware problem. @oleg-plakida can you maybe post some SWC links from the past VM+BM runs to demonstrate?

Here is one for BM with some fluctuations.
Here is the vm one with similar results.

@bkchr
Copy link
Member

bkchr commented May 19, 2023

The only noise we still see is in 3-5 benchmarks (two of them in pallet-contracts) consistently, but that is also present on bm3. So I assume that these benches are unstable themselves, and not a hardware problem.

What happened to paritytech/polkadot-sdk#379?

@ggwpez
Copy link
Member

ggwpez commented May 19, 2023

The only noise we still see is in 3-5 benchmarks (two of them in pallet-contracts) consistently, but that is also present on bm3. So I assume that these benches are unstable themselves, and not a hardware problem.

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.
For example batch. I ran it on bm2 on the last 10 master commits (this was a month back or so) and 10 times on each commit with 2000 reps. WASM shows consistent outliers - although nothing in the code of batch changed. Dont know if it is some WASM memory alignment or other oddities that screw it.

This lead me to de-prioritize the dynamic reps, as it wont solve that fluctuation.
wasm
Native looks a bit better:
native

Copy link
Member

@ggwpez ggwpez left a 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.

@mateo-moon
Copy link
Contributor Author

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.

@koute
Copy link
Contributor

koute commented Jun 5, 2023

Problem is that even with very high repetitions (2000 instead of 50), some calls show consistent outlier behaviour.

@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 ruby, setarch and schedtool are installed, download the script, chmod +x it, and run ./perftest.rb $command.)

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.)

@mateo-moon mateo-moon added D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit and removed D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. labels Jun 8, 2023
@paritytech paritytech deleted a comment from command-bot bot Jun 8, 2023
@paritytech paritytech deleted a comment from command-bot bot Jun 8, 2023
@paritytech paritytech deleted a comment from paritytech-processbot bot Jun 8, 2023
@mateo-moon mateo-moon closed this Jun 8, 2023
@mateo-moon mateo-moon force-pushed the oleg/ci-cd/vm_weights_baseline branch from cb204bf to 85b0807 Compare June 8, 2023 13:20
@mateo-moon mateo-moon reopened this Jun 9, 2023
@mateo-moon mateo-moon requested a review from a team June 9, 2023 10:44
@mateo-moon mateo-moon force-pushed the oleg/ci-cd/vm_weights_baseline branch from f67bc8e to c9c84df Compare June 16, 2023 14:58
@mateo-moon mateo-moon closed this Jun 16, 2023
@mateo-moon mateo-moon force-pushed the oleg/ci-cd/vm_weights_baseline branch from c9c84df to 6bee6fc Compare June 16, 2023 15:15
@mateo-moon
Copy link
Contributor Author

bot bench-vm $ all

@paritytech paritytech deleted a comment from paritytech-cicd-pr Jun 16, 2023
@mateo-moon mateo-moon reopened this Jun 18, 2023
@mateo-moon mateo-moon merged commit a626589 into master Jun 19, 2023
@mateo-moon mateo-moon deleted the oleg/ci-cd/vm_weights_baseline branch June 19, 2023 13:31
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants