Closed mtauraso closed 1 month ago
Before [71af541d] | After [ee4b46ef] | Ratio | Benchmark (Parameter) |
---|---|---|---|
151±1ms | 152±1ms | 1.01 | benchmarks.time_learn_loop('KNN', 'RandomSampling') |
196M | 197M | 1 | benchmarks.peakmem_learn_loop('KNN') |
188M | 188M | 1 | benchmarks.peakmem_learn_loop('RandomForest') |
154±2ms | 154±3ms | 1 | benchmarks.time_learn_loop('KNN', 'UncSampling') |
2.58±0.01s | 2.59±0.01s | 1 | benchmarks.time_learn_loop('RandomForest', 'RandomSampling') |
135±2ms | 134±1ms | 0.99 | benchmarks.time_feature_creation |
2.59±0s | 2.58±0.03s | 0.99 | benchmarks.time_learn_loop('RandomForest', 'UncSampling') |
Click here to view all benchmarks.
Based on slack conversation with @AmandaWasserman and @rknop I'm overriding the requirement for review and merging this pull request.