Skip to content

Commit

Permalink
FileSizeBuckets in correct units (dotnet#387)
Browse files Browse the repository at this point in the history
* Minor telemetry change to log in correct units and make our life easier in the future

* Use Ceiling instead of Round
  • Loading branch information
justinormont authored Apr 12, 2019
1 parent c5ec302 commit d4b7700
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/mlnet/Telemetry/MlTelemetry.cs
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ public void LogAutoTrainMlCommand(string dataFileName, string task, long dataFil

var telemetry = new Telemetry();

var fileSizeBucket = Math.Ceiling(Math.Log(dataFileSize, 2));
var fileSizeBucket = Math.Pow(2, Math.Ceiling(Math.Log(dataFileSize, 2)));

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

Expand Down

0 comments on commit d4b7700

Please sign in to comment.