-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[Perf] Changes at 5/19/2022 4:33:45 PM #69724
Comments
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
@DrewScoggins - Seems we didn't have run for few days. Can we do catchup runs to narrow down the commit that caused the regression? |
Assigning to Extensions label until we can get an updated commit range. |
Tagging subscribers to this area: @dotnet/area-extensions-logging Issue DetailsRun Information
Regressions in Microsoft.Extensions.Logging.FormattingOverhead
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'Microsoft.Extensions.Logging.FormattingOverhead*' PayloadsHistogramMicrosoft.Extensions.Logging.FormattingOverhead.TwoArguments_DefineMessage
Description of detection logic
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository
Regressions in Microsoft.Extensions.Logging.Formatting
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'Microsoft.Extensions.Logging.Formatting*' PayloadsHistogramMicrosoft.Extensions.Logging.Formatting.TwoArguments
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository Run Information
Regressions in System.Tests.Perf_String
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'System.Tests.Perf_String*' PayloadsHistogramSystem.Tests.Perf_String.Format_MultipleArgs
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository Run Information
Regressions in System.Globalization.Tests.StringEquality
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'System.Globalization.Tests.StringEquality*' PayloadsHistogramSystem.Globalization.Tests.StringEquality.Compare_Same(Count: 1024, Options: (en-US, OrdinalIgnoreCase))
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository Run Information
Regressions in System.IO.Tests.Perf_StreamWriter
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'System.IO.Tests.Perf_StreamWriter*' PayloadsHistogramSystem.IO.Tests.Perf_StreamWriter.WriteFormat
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository
|
More regression data for System.Globalization.Tests.StringEquality.Compare_Same(Count: 1024, Options: (en-US, OrdinalIgnoreCase))
|
More regression data for System.Tests.Perf_String.Format_MultipleArgs
|
Looking at the list of commits 6e4eba8...15dda0e I am not seeing any commit for logging nor globalization which can cause any regression. @AndyAyersMS do you think 41491a3 can cause such regression? |
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch Issue DetailsRun Information
Regressions in Microsoft.Extensions.Logging.FormattingOverhead
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'Microsoft.Extensions.Logging.FormattingOverhead*' PayloadsHistogramMicrosoft.Extensions.Logging.FormattingOverhead.TwoArguments_DefineMessage
Description of detection logic
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository
Regressions in Microsoft.Extensions.Logging.Formatting
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'Microsoft.Extensions.Logging.Formatting*' PayloadsHistogramMicrosoft.Extensions.Logging.Formatting.TwoArguments
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository Run Information
Regressions in System.Tests.Perf_String
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'System.Tests.Perf_String*' PayloadsHistogramSystem.Tests.Perf_String.Format_MultipleArgs
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository Run Information
Regressions in System.Globalization.Tests.StringEquality
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'System.Globalization.Tests.StringEquality*' PayloadsHistogramSystem.Globalization.Tests.StringEquality.Compare_Same(Count: 1024, Options: (en-US, OrdinalIgnoreCase))
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository Run Information
Regressions in System.IO.Tests.Perf_StreamWriter
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'System.IO.Tests.Perf_StreamWriter*' PayloadsHistogramSystem.IO.Tests.Perf_StreamWriter.WriteFormat
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository
|
No, it was a bug fix for a very rare case. Seems like whatever the cause of these regressions, most now are faster now -- perhaps we can just close this. |
Per @AndyAyersMS reply, I am closing this issue for now as the regression is not there anymore. Thanks @AndyAyersMS! |
Run Information
Regressions in Microsoft.Extensions.Logging.FormattingOverhead
Test Report
Repro
Payloads
Baseline
Compare
Histogram
Microsoft.Extensions.Logging.FormattingOverhead.TwoArguments_DefineMessage
Description of detection logic
Description of detection logic
Docs
Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository
Regressions in Microsoft.Extensions.Logging.Formatting
Test Report
Repro
Payloads
Baseline
Compare
Histogram
Microsoft.Extensions.Logging.Formatting.TwoArguments
Description of detection logic
Docs
Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository
Run Information
Regressions in System.Tests.Perf_String
Test Report
Repro
Payloads
Baseline
Compare
Histogram
System.Tests.Perf_String.Format_MultipleArgs
Description of detection logic
Docs
Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository
Run Information
Regressions in System.Globalization.Tests.StringEquality
Test Report
Repro
Payloads
Baseline
Compare
Histogram
System.Globalization.Tests.StringEquality.Compare_Same(Count: 1024, Options: (en-US, OrdinalIgnoreCase))
Description of detection logic
Docs
Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository
Run Information
Regressions in System.IO.Tests.Perf_StreamWriter
Test Report
Repro
Payloads
Baseline
Compare
Histogram
System.IO.Tests.Perf_StreamWriter.WriteFormat
Description of detection logic
Docs
Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository
The text was updated successfully, but these errors were encountered: