Skip to content

Commit 46c36dc

Browse files
enable BinaryClassifierSymSgdTest (#4863)
* enable BinaryClassifierSymSgdTest * add TODO tag for linux and macos cases so we can easily go back to issue
1 parent b7b9569 commit 46c36dc

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

test/Microsoft.ML.Predictor.Tests/TestPredictors.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -276,16 +276,14 @@ public void BinaryClassifierLogisticRegressionTest()
276276

277277
[X64Fact("x86 output differs from Baseline")]
278278
[TestCategory("Binary")]
279-
//Skipping test temporarily. This test will be re-enabled once the cause of failures has been determined
280-
[Trait("Category", "SkipInCI")]
281279
public void BinaryClassifierSymSgdTest()
282280
{
283-
// Linux uses a version of MKL that doesn't support conditional numerical reproducibility the same way as
281+
// TODO: Linux uses a version of MKL that doesn't support conditional numerical reproducibility the same way as
284282
// Windows runs.
285283
if (RuntimeInformation.IsOSPlatform(OSPlatform.Linux))
286284
return;
287285

288-
//Results sometimes go out of error tolerance on OS X.
286+
// TODO: Results sometimes go out of error tolerance on OS X.
289287
if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX))
290288
return;
291289

0 commit comments

Comments
 (0)