Skip to content

Commit 09c9adf

Browse files
justinormontDmitry-A
authored andcommitted
FileSizeBuckets in correct units (dotnet#387)
* Minor telemetry change to log in correct units and make our life easier in the future * Use Ceiling instead of Round
1 parent 3056e3f commit 09c9adf

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/mlnet/Telemetry/MlTelemetry.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public void LogAutoTrainMlCommand(string dataFileName, string task, long dataFil
3535

3636
var telemetry = new Telemetry();
3737

38-
var fileSizeBucket = Math.Ceiling(Math.Log(dataFileSize, 2));
38+
var fileSizeBucket = Math.Pow(2, Math.Ceiling(Math.Log(dataFileSize, 2)));
3939

4040
var fileNameHash = string.IsNullOrEmpty(dataFileName) ? string.Empty : Sha256Hasher.Hash(dataFileName);
4141

0 commit comments

Comments
 (0)