Avoid unsealed attributes (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1813)

Possible multiple enumerations of 'IEnumerable' collection. Consider using an implementation that avoids multiple enumerations. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1851)

Possible multiple enumerations of 'IEnumerable' collection. Consider using an implementation that avoids multiple enumerations. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1851)

Exception type System.Exception is not sufficiently specific (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2201)

Prefer a 'TryGetValue' call over a Dictionary indexer access guarded by a 'ContainsKey' check to avoid double lookup (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1854)

Exception type System.Exception is not sufficiently specific (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2201)

'active == null' is always 'true'. Remove or refactor the condition(s) to avoid dead code. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1508)

In member bool MetricsCollection.Contains(Metric metric), change parameter name metric to item in order to match the identifier as it has been declared in bool ICollection<Metric>.Contains(Metric item) (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1725)

In member bool MetricsCollection.Remove(Metric metric), change parameter name metric to item in order to match the identifier as it has been declared in bool ICollection<Metric>.Remove(Metric item) (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1725)