Skip to content

Ensure the named model is loaded#5833

Merged
michaelgsharp merged 1 commit intodotnet:mainfrom
feiyun0112:patch15
Jun 3, 2021
Merged

Ensure the named model is loaded#5833
michaelgsharp merged 1 commit intodotnet:mainfrom
feiyun0112:patch15

Conversation

@feiyun0112
Copy link
Contributor

Fix #5831

using System.IO;
using System.Threading;
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Logging;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

I think you have some redundant imports here.

@codecov
Copy link

codecov bot commented Jun 3, 2021

Codecov Report

Merging #5833 (76c72c9) into main (d266c86) will increase coverage by 0.03%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main    #5833      +/-   ##
==========================================
+ Coverage   68.32%   68.36%   +0.03%     
==========================================
  Files        1131     1132       +1     
  Lines      241291   241313      +22     
  Branches    25053    25054       +1     
==========================================
+ Hits       164863   164970     +107     
+ Misses      69923    69839      -84     
+ Partials     6505     6504       -1     
Flag Coverage Δ
Debug 68.36% <83.33%> (+0.03%) ⬆️
production 62.98% <60.00%> (+0.04%) ⬆️
test 89.24% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...rc/Microsoft.Extensions.ML/PredictionEnginePool.cs 37.70% <60.00%> (+37.70%) ⬆️
...t.Extensions.ML.Tests/PredictionEnginePoolTests.cs 100.00% <100.00%> (ø)
...rosoft.ML.AutoML/ColumnInference/TextFileSample.cs 62.25% <0.00%> (-3.32%) ⬇️
...StandardTrainers/Standard/LinearModelParameters.cs 66.32% <0.00%> (+0.25%) ⬆️
...soft.ML.Data/DataLoadSave/Text/TextLoaderCursor.cs 90.41% <0.00%> (+0.63%) ⬆️
...soft.ML.Transforms/Text/WordEmbeddingsExtractor.cs 86.87% <0.00%> (+1.13%) ⬆️
src/Microsoft.ML.Sweeper/AsyncSweeper.cs 72.78% <0.00%> (+1.36%) ⬆️
....ML.AutoML/PipelineSuggesters/PipelineSuggester.cs 81.88% <0.00%> (+3.14%) ⬆️
... and 7 more

Copy link
Contributor

@michaelgsharp michaelgsharp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for the submission.

@michaelgsharp michaelgsharp merged commit 8374401 into dotnet:main Jun 3, 2021
@feiyun0112 feiyun0112 deleted the patch15 branch June 4, 2021 08:03
@ghost ghost locked as resolved and limited conversation to collaborators Mar 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Microsoft.Extensions.ML] PredictionEnginePool.GetModel(string modelName) does not ensure the named model is loaded

3 participants