search

dotnet / machinelearning

ML.NET is an open source and cross-platform machine learning framework for .NET.
https://dot.net/ml
MIT License
8.92k stars 1.86k forks source link

[release/3.0] Split out non concurrent test collections. #6950

Closed michaelgsharp closed 5 months ago

michaelgsharp commented 5 months ago

Backport of #6937 to release/3.0. Fixed version of closed pr #6947

/cc @michaelgsharp

Customer Impact

Improves the testing process for PR's to better facilitate us getting PR's out.

Testing

All tests pass with new changes and only test assemblies are changed.

Risk

None, only test assemblies are changed.

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f602581) 68.80% compared to head (a4c578d) 68.80%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## release/3.0 #6950 +/- ## ============================================ Coverage 68.80% 68.80% ============================================ Files 1240 1240 Lines 249604 249610 +6 Branches 25507 25507 ============================================ + Hits 171740 171755 +15 + Misses 71265 71257 -8 + Partials 6599 6598 -1 ``` | [Flag](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | Coverage Δ | | |---|---|---| | [Debug](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | `68.80% <100.00%> (+<0.01%)` | :arrow_up: | | [production](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | `63.27% <ø> (+<0.01%)` | :arrow_up: | | [test](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | `88.41% <100.00%> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | Coverage Δ | | |---|---|---| | [...ft.ML.Core.Tests/UnitTests/TestResourceDownload.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-dGVzdC9NaWNyb3NvZnQuTUwuQ29yZS5UZXN0cy9Vbml0VGVzdHMvVGVzdFJlc291cmNlRG93bmxvYWQuY3M=) | `0.00% <ø> (ø)` | | | [...ft.ML.TensorFlow.Tests/TensorFlowEstimatorTests.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-dGVzdC9NaWNyb3NvZnQuTUwuVGVuc29yRmxvdy5UZXN0cy9UZW5zb3JGbG93RXN0aW1hdG9yVGVzdHMuY3M=) | `98.50% <ø> (ø)` | | | [...t/Microsoft.ML.TensorFlow.Tests/TensorflowTests.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-dGVzdC9NaWNyb3NvZnQuTUwuVGVuc29yRmxvdy5UZXN0cy9UZW5zb3JmbG93VGVzdHMuY3M=) | `91.71% <ø> (ø)` | | | [test/Microsoft.ML.Tests/DatabaseLoaderTests.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-dGVzdC9NaWNyb3NvZnQuTUwuVGVzdHMvRGF0YWJhc2VMb2FkZXJUZXN0cy5jcw==) | `94.41% <100.00%> (+0.17%)` | :arrow_up: | | [test/Microsoft.ML.TorchSharp.Tests/NerTests.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-dGVzdC9NaWNyb3NvZnQuTUwuVG9yY2hTaGFycC5UZXN0cy9OZXJUZXN0cy5jcw==) | `67.02% <ø> (ø)` | | | [...rosoft.ML.TorchSharp.Tests/ObjectDetectionTests.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-dGVzdC9NaWNyb3NvZnQuTUwuVG9yY2hTaGFycC5UZXN0cy9PYmplY3REZXRlY3Rpb25UZXN0cy5jcw==) | `100.00% <ø> (ø)` | | | [test/Microsoft.ML.TorchSharp.Tests/QATests.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-dGVzdC9NaWNyb3NvZnQuTUwuVG9yY2hTaGFycC5UZXN0cy9RQVRlc3RzLmNz) | `40.50% <ø> (ø)` | | | [...oft.ML.TorchSharp.Tests/TextClassificationTests.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-dGVzdC9NaWNyb3NvZnQuTUwuVG9yY2hTaGFycC5UZXN0cy9UZXh0Q2xhc3NpZmljYXRpb25UZXN0cy5jcw==) | `85.17% <ø> (ø)` | | ... and [3 files with indirect coverage changes](https://app.codecov.io/gh/dotnet/machinelearning/pull/6950/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet)