Skip to content

Commit 7e21afa

Browse files
authored
using helper method instead of cleanup (#1946)
1 parent a570da1 commit 7e21afa

File tree

3 files changed

+30
-14
lines changed

3 files changed

+30
-14
lines changed
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
// Licensed to the .NET Foundation under one or more agreements.
2+
// The .NET Foundation licenses this file to you under the MIT license.
3+
// See the LICENSE file in the project root for more information.
4+
5+
using Microsoft.ML.Tools;
6+
using System;
7+
8+
namespace Microsoft.ML.Benchmarks
9+
{
10+
internal static class ExecuteMaml
11+
{
12+
public static void ExecuteMamlCommand(this string command, MLContext environment)
13+
{
14+
if (Maml.MainCore(environment, command, alwaysPrintStacktrace: false) < 0)
15+
throw new Exception($"Command {command} returned negative error code");
16+
}
17+
}
18+
}

test/Microsoft.ML.Benchmarks/Numeric/Ranking.cs

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,13 @@
22
// The .NET Foundation licenses this file to you under the MIT license.
33
// See the LICENSE file in the project root for more information.
44

5-
using System.IO;
65
using BenchmarkDotNet.Attributes;
76
using Microsoft.ML.Data;
87
using Microsoft.ML.LightGBM;
98
using Microsoft.ML.RunTests;
10-
using Microsoft.ML.Tools;
119
using Microsoft.ML.Trainers.FastTree;
1210
using Microsoft.ML.Transforms.Conversions;
11+
using System.IO;
1312

1413
namespace Microsoft.ML.Benchmarks
1514
{
@@ -43,7 +42,7 @@ public void TrainTest_Ranking_MSLRWeb10K_RawNumericFeatures_FastTreeRanking()
4342
" tr=FastTreeRanking{}";
4443

4544
var environment = EnvironmentFactory.CreateRankingEnvironment<RankerEvaluator, TextLoader, HashingTransformer, FastTreeRankingTrainer>();
46-
Maml.MainCore(environment, cmd, alwaysPrintStacktrace: false);
45+
cmd.ExecuteMamlCommand(environment);
4746
}
4847

4948
[Benchmark]
@@ -58,7 +57,7 @@ public void TrainTest_Ranking_MSLRWeb10K_RawNumericFeatures_LightGBMRanking()
5857
" tr=LightGBMRanking{}";
5958

6059
var environment = EnvironmentFactory.CreateRankingEnvironment<RankerEvaluator, TextLoader, HashingTransformer, LightGbmMulticlassTrainer>();
61-
Maml.MainCore(environment, cmd, alwaysPrintStacktrace: false);
60+
cmd.ExecuteMamlCommand(environment);
6261
}
6362
}
6463

@@ -97,7 +96,7 @@ public void SetupScoringSpeedTests()
9796
" out={" + _modelPath_MSLR + "}";
9897

9998
var environment = EnvironmentFactory.CreateRankingEnvironment<RankerEvaluator, TextLoader, HashingTransformer, FastTreeRankingTrainer>();
100-
Maml.MainCore(environment, cmd, alwaysPrintStacktrace: false);
99+
cmd.ExecuteMamlCommand(environment);
101100
}
102101

103102
[Benchmark]
@@ -107,7 +106,7 @@ public void Test_Ranking_MSLRWeb10K_RawNumericFeatures_FastTreeRanking()
107106
string cmd = @"Test data=" + _mslrWeb10k_Test + " in=" + _modelPath_MSLR;
108107

109108
var environment = EnvironmentFactory.CreateRankingEnvironment<RankerEvaluator, TextLoader, HashingTransformer, FastTreeRankingTrainer>();
110-
Maml.MainCore(environment, cmd, alwaysPrintStacktrace: false);
109+
cmd.ExecuteMamlCommand(environment);
111110
}
112111
}
113112
}

test/Microsoft.ML.Benchmarks/Text/MultiClassClassification.cs

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,14 @@
22
// The .NET Foundation licenses this file to you under the MIT license.
33
// See the LICENSE file in the project root for more information.
44

5-
using System.IO;
65
using BenchmarkDotNet.Attributes;
76
using Microsoft.ML.Data;
87
using Microsoft.ML.LightGBM;
98
using Microsoft.ML.RunTests;
10-
using Microsoft.ML.Tools;
119
using Microsoft.ML.Trainers;
1210
using Microsoft.ML.Trainers.Online;
1311
using Microsoft.ML.Transforms.Categorical;
12+
using System.IO;
1413

1514
namespace Microsoft.ML.Benchmarks
1615
{
@@ -40,7 +39,7 @@ public void CV_Multiclass_WikiDetox_BigramsAndTrichar_OVAAveragedPerceptron()
4039
" tr=OVA{p=AveragedPerceptron{iter=10}}";
4140

4241
var environment = EnvironmentFactory.CreateClassificationEnvironment<TextLoader, OneHotEncodingTransformer, AveragedPerceptronTrainer>();
43-
Maml.MainCore(environment, cmd, alwaysPrintStacktrace: false);
42+
cmd.ExecuteMamlCommand(environment);
4443
}
4544

4645
[Benchmark]
@@ -55,7 +54,7 @@ public void CV_Multiclass_WikiDetox_BigramsAndTrichar_LightGBMMulticlass()
5554
" tr=LightGBMMulticlass{iter=10}";
5655

5756
var environment = EnvironmentFactory.CreateClassificationEnvironment<TextLoader, OneHotEncodingTransformer, LightGbmMulticlassTrainer>();
58-
Maml.MainCore(environment, cmd, alwaysPrintStacktrace: false);
57+
cmd.ExecuteMamlCommand(environment);
5958
}
6059

6160
[Benchmark]
@@ -71,7 +70,7 @@ public void CV_Multiclass_WikiDetox_WordEmbeddings_OVAAveragedPerceptron()
7170
" xf=Concat{col=Features:FeaturesText,FeaturesWordEmbedding,logged_in,ns}";
7271

7372
var environment = EnvironmentFactory.CreateClassificationEnvironment<TextLoader, OneHotEncodingTransformer, AveragedPerceptronTrainer>();
74-
Maml.MainCore(environment, cmd, alwaysPrintStacktrace: false);
73+
cmd.ExecuteMamlCommand(environment);
7574
}
7675

7776
[Benchmark]
@@ -87,7 +86,7 @@ public void CV_Multiclass_WikiDetox_WordEmbeddings_SDCAMC()
8786
" xf=Concat{col=Features:FeaturesWordEmbedding,logged_in,ns}";
8887

8988
var environment = EnvironmentFactory.CreateClassificationEnvironment<TextLoader, OneHotEncodingTransformer, SdcaMultiClassTrainer>();
90-
Maml.MainCore(environment, cmd, alwaysPrintStacktrace: false);
89+
cmd.ExecuteMamlCommand(environment);
9190
}
9291
}
9392

@@ -115,7 +114,7 @@ public void SetupScoringSpeedTests()
115114
" out={" + _modelPath_Wiki + "}";
116115

117116
var environment = EnvironmentFactory.CreateClassificationEnvironment<TextLoader, OneHotEncodingTransformer, AveragedPerceptronTrainer>();
118-
Maml.MainCore(environment, cmd, alwaysPrintStacktrace: false);
117+
cmd.ExecuteMamlCommand(environment);
119118
}
120119

121120
[Benchmark]
@@ -126,7 +125,7 @@ public void Test_Multiclass_WikiDetox_BigramsAndTrichar_OVAAveragedPerceptron()
126125
string cmd = @"Test data=" + _dataPath_Wiki + " in=" + modelpath;
127126

128127
var environment = EnvironmentFactory.CreateClassificationEnvironment<TextLoader, OneHotEncodingTransformer, AveragedPerceptronTrainer>();
129-
Maml.MainCore(environment, cmd, alwaysPrintStacktrace: false);
128+
cmd.ExecuteMamlCommand(environment);
130129
}
131130
}
132131
}

0 commit comments

Comments
 (0)