Feature Request: Add ability to automatically hide metric columns if value is not set

[filzrev]

When using ThreadingDiagnoser/ExceptionDiagnoser.
Following metric columns are displayed always .

• Exceptions
• Completed Work Items
• Lock Contentions

Is it able to automatically hide these columns if all metric values are 0 ?.

It seems to be able to change behaviors by modifying following code.

• MetricColumn::IsAvailable
• IMetricDescriptor::GetIsAvailable

Excample benchmark code

 [ShortRunJob]
 [ThreadingDiagnoser]
 // [HideColumns(Column.CompletedWorkItems, Column.LockContentions, Column.Exceptions)]
 public class Benchmarks
 {
     [Benchmark]
     public void Benchmark()
     {
         Thread.Sleep(1);
     }
 }

Console Output

| Method    | Mean     | Error    | StdDev   | Exceptions | Completed Work Items | Lock Contentions |
|---------- |---------:|---------:|---------:|-----------:|---------------------:|-----------------:|
| Benchmark | 15.57 ms | 0.364 ms | 0.020 ms |          - |                    - |                - |

[timcassell]

It should be fairly easy to implement that behavior by changing the implementation of GetIsAvailable for those metric descriptors, similar to MemoryDiagnoser.

BenchmarkDotNet/src/BenchmarkDotNet/Diagnosers/MemoryDiagnoser.cs

Line 69 in ca5dfdf

public bool GetIsAvailable(Metric metric) => metric.Value > 0;

BenchmarkDotNet/src/BenchmarkDotNet/Diagnosers/ThreadingDiagnoser.cs

Line 65 in ca5dfdf

public bool GetIsAvailable(Metric metric) => true;

BenchmarkDotNet/src/BenchmarkDotNet/Diagnosers/ThreadingDiagnoser.cs

Line 80 in ca5dfdf

public bool GetIsAvailable(Metric metric) => true;

BenchmarkDotNet/src/BenchmarkDotNet/Diagnosers/ExceptionDiagnoser.cs

Line 52 in ca5dfdf

public bool GetIsAvailable(Metric metric) => true;

[AvishaiDotan]

First and foremost: This is my first contribution - pretty exciting 😊

I Followed @timcassell instructions and PR this:

#2673