Skip to content
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

Something went wrong with outliers when using --launchCount #2432

Closed
AndyAyersMS opened this issue Sep 18, 2023 · 4 comments
Closed

Something went wrong with outliers when using --launchCount #2432

AndyAyersMS opened this issue Sep 18, 2023 · 4 comments
Assignees
Milestone

Comments

@AndyAyersMS
Copy link
Member

I have been playing around with --launchCount and noticed this when looking at the data:

BenchmarkDotNet v0.13.7-nightly.20230717.35, Windows 11 (10.0.22621.2283/22H2/2022Update/SunValley2)
Intel Core i7-8700 CPU 3.20GHz (Coffee Lake), 1 CPU, 12 logical and 6 physical cores
.NET SDK 8.0.100-preview.7.23376.3
[Host] : .NET 6.0.22 (6.0.2223.42425), X64 RyuJIT AVX2
Job-QYHTSN : .NET 9.0.0 (42.42.42.42424), X64 RyuJIT AVX2

EnvironmentVariables=DOTNET_JitCSEHash=26ab038f,DOTNET_JitDisasmSummary=1,DOTNET_JitStdOutFile=D:\bugs\cse\base.4a.list,DOTNET_JitCSEMask=4 PowerPlanMode=00000000-0000-0000-0000-000000000000 Arguments=/p:EnableUnsafeBinaryFormatterSerialization=true
Toolchain=CoreRun IterationTime=250.0000 ms LaunchCount=100
MaxIterationCount=20 MinIterationCount=15 WarmupCount=1

Method Count Mean Error StdDev Median Min Max Gen0 Gen1 Allocated
Dictionary 512 13.71 μs 0.052 μs 0.598 μs 13.82 μs 12.44 μs 17.10 μs 7.6273 0.7133 46.97 KB

// * Hints *
Outliers
TryAddDefaultSize.Dictionary: EnvironmentVariables=DOTNET_JitCSEHash=26ab038f,DOTNET_JitDisasmSummary=1,DOTNET_JitStdOutFile=D:\bugs\cse\base.4a.list,DOTNET_JitCSEMask=4, PowerPlanMode=00000000-0000-0000-0000-000000000000, Arguments=/p:EnableUnsafeBinaryFormatterSerialization=true, Toolchain=CoreRun, IterationTime=250.0000 ms, LaunchCount=100, MaxIterationCount=20, MinIterationCount=15, WarmupCount=1 -> Something went wrong with outliers: Size(WorkloadActual) = 1548, Size(WorkloadActual/Outliers) = 34, Size(Result) = 1467), OutlierMode = RemoveUpper

This was running --filter "System.Collections.TryAddDefaultSize<String>.Dictionary" from dotnet/performance using their config setup and a locally built corerun. But I suspect it will repro with a normal build.

@AndyAyersMS AndyAyersMS changed the title Something went wrong with outliers when using --LaunchCount Something went wrong with outliers when using --launchCount Sep 18, 2023
@AndreyAkinshin
Copy link
Member

@AndyAyersMS thanks for the report and sorry for the late reply. Could you please share the full log from the run that reproduces this problem?

@AndreyAkinshin AndreyAkinshin self-assigned this Oct 3, 2023
@AndreyAkinshin
Copy link
Member

AndreyAkinshin commented Oct 3, 2023

@AndyAyersMS no logs needed, I have been able to reproduce the problem.

@AndyAyersMS
Copy link
Member Author

@AndyAyersMS no logs needed, I have been able to reproduce the problem.

That's good because I likely don't have it anymore.

@AndreyAkinshin
Copy link
Member

@AndyAyersMS it turned out that this check was incorrect, so I just removed it. Thanks again for the report!

@AndreyAkinshin AndreyAkinshin added this to the v0.13.9 milestone Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants