-
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
Regressions in System.IO.Tests.StreamReaderReadLineTests #78176
Comments
Tagging subscribers to this area: @dotnet/area-system-io Issue DetailsRun Information
Regressions in System.IO.Tests.StreamReaderReadLineTests
Reprogit clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net6.0 --filter 'System.IO.Tests.StreamReaderReadLineTests*' Related IssuesRegressionsImprovementsPayloadsHistogramEdge Detector InfoSystem.IO.Tests.StreamReaderReadLineTests.ReadLineAsync(LineLengthRange: [ 1, 1])
Description of detection logic
DocsProfiling workflow for dotnet/runtime repository
|
Likely regressed from #69888 cc @GrabYourPitchforks |
It's by design: #69888 has improved a lot of test cases: dotnet/perf-autofiling-issues#9681 but regressed this one, which is about reading very short lines (1 character). I think it's acceptable. cc @stephentoub |
Yup. We had a similar conversation and outcome when similar changes to use IndexOf{Any} were made to StringReader. |
Run Information
Regressions in System.IO.Tests.StreamReaderReadLineTests
Test Report
Repro
Related Issues
Regressions
Improvements
Payloads
Baseline
Compare
Histogram
Edge Detector Info
Collection Data
System.IO.Tests.StreamReaderReadLineTests.ReadLineAsync(LineLengthRange: [ 1, 1])
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: