-
-
Notifications
You must be signed in to change notification settings - Fork 969
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
[DOC] Wasm doc needs update #1818
Comments
Thanks for the initiative @naricc . I would love to have some inspiration on how to compare different versions of wasm runtimes (i.e. .net 5 vs .net 6 wasm, and perhaps even aot wasm) |
@radical - sending this your way, as many changes have been done for wasm related pipeline which needs documentation. |
I wanted to benchmark WASM AOT today to unblock dotnet/runtime#73556 (comment)
--runtimes wasm --AOTCompilerMode wasm --wasmDataDir /home/adam/projects/runtime/src/mono/wasm/test-main.js Which gave me following errors:
I can see that BDN tries to perform I've tried to install the workload manually: cd /home/adam/projects/performance/artifacts/bin/MicroBenchmarks/Release/net7.0/27456a6c-b7a5-4af8-8816-6675f6cb0439/
/home/adam/projects/performance/tools/dotnet/x64/dotnet workload restore --packages "/home/adam/projects/performance/artifacts/packages" /p:UseSharedCompilation=false /p:BuildInParallel=false /m:1 /p:Deterministic=true /p:Optimize=true --project ./BenchmarkDotNet.Autogenerated.csproj And got new failures:
The |
This is needed -
This is what I have been using, and based it on the CI runs. |
It's already in the |
@radical thanks for sharing your steps! I got to the point where I was getting:
but I just commented out this line of FWIW the command that I've used: python3 ./scripts/benchmarks_ci.py --filter '*Burgers.Test0' -f net7.0 --bdn-arguments "--runtimes wasmnet7.0 --wasmDataDir /home/adam/projects/repros/wasm/" |
This document and assocaited source code need updating, since --wasmMainJS and --customruntimepack options are no longer needed, and the --runtimeSrcDir argument is now required:
https://github.com/dotnet/BenchmarkDotNet/blob/master/docs/articles/samples/IntroWasm.md/#L1
I intend to take on this task; creating the issue so I can keep track of things.
The text was updated successfully, but these errors were encountered: