Improvements (#65)

tonyredondo · Nov 26, 2024 · ed6d965 · ed6d965
1 parent 1b56e3e
commit ed6d965
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/src/Directory.Build.props b/src/Directory.Build.props
@@ -1,6 +1,6 @@
 <Project>
     <PropertyGroup>
-        <Version>0.3.0</Version>
+        <Version>0.3.1</Version>
         <Authors>Tony Redondo, Grégory Léocadie</Authors>
         <TargetFrameworks>net6.0;net7.0;net8.0;net9.0</TargetFrameworks>
         <ImplicitUsings>enable</ImplicitUsings>

diff --git a/src/TimeItSharp.Common/Configuration/Config.cs b/src/TimeItSharp.Common/Configuration/Config.cs
@@ -91,7 +91,7 @@ public Config()
         ProcessFailedDataPoints = false;
         ShowStdOutForFirstRun = false;
         DebugMode = false;
-        AcceptableRelativeWidth = 0.01;
+        AcceptableRelativeWidth = 0.007;
         ConfidenceLevel = 0.95;
         MaximumDurationInMinutes = 45;
         EvaluationInterval = 10;

diff --git a/src/TimeItSharp.Common/Exporters/ConsoleExporter.cs b/src/TimeItSharp.Common/Exporters/ConsoleExporter.cs
@@ -141,7 +141,7 @@ public void Export(TimeitResult results)
                 var outliersValue = result.Outliers.Count > 0 ? $"{result.Outliers.Count} {{{Math.Round(result.OutliersThreshold, 3)}}}" : "0";
                 var rowList = new List<string>
                 {
-                    $"[aqua underline]{result.Name}[/]",
+                    $"[aqua underline]{result.Name} [[N={result.Count}]][/]",
                     $"{(result.Status == Status.Passed ? "[aqua]Passed" : "[red]Failed")}[/]",
                     $"[aqua]{Math.Round(Utils.FromNanosecondsToMilliseconds(result.Mean), 3)}ms[/]",
                     $"[aqua]{Math.Round(Utils.FromNanosecondsToMilliseconds(result.Stdev), 3)}ms[/]",

diff --git a/src/TimeItSharp.Common/ScenarioProcessor.cs b/src/TimeItSharp.Common/ScenarioProcessor.cs
@@ -382,7 +382,7 @@ await RunScenarioAsync(repeat.Count, index, scenario, TimeItPhase.ExtraRun, fals
         var scenarioResult = new ScenarioResult
         {
             Scenario = scenario,
-            Count = _configuration.Count,
+            Count = dataPoints.Count,
             WarmUpCount = _configuration.WarmUpCount,
             Data = dataPoints,
             Durations = newDurations,
@@ -426,7 +426,7 @@ await RunScenarioAsync(repeat.Count, index, scenario, TimeItPhase.ExtraRun, fals
 
     private async Task<List<DataPoint>> RunScenarioAsync(int count, int index, Scenario scenario, TimeItPhase phase, bool checkShouldContinue, Stopwatch stopwatch, CancellationToken cancellationToken)
     {
-        var minIterations = count / 3;
+        var minIterations = count / 2.5;
         minIterations = minIterations < 10 ? 10 : minIterations;
         var confidenceLevel = _configuration.ConfidenceLevel;
         if (confidenceLevel is <= 0 or >= 1)
@@ -529,7 +529,7 @@ static double GetDuration(DataPoint point)
                         if ((durations.Count - minIterations) % _configuration.EvaluationInterval == 0)
                         {
                             var errorReduction = (previousRelativeWidth - relativeWidth) / previousRelativeWidth;
-                            if (errorReduction < _configuration.MinimumErrorReduction)
+                            if (errorReduction > 0 && errorReduction < _configuration.MinimumErrorReduction)
                             {
                                 AnsiConsole.WriteLine();
                                 AnsiConsole.MarkupLine(