Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix DNN Featurizer sample bug. #3237

Merged
merged 1 commit into from
Apr 8, 2019
Merged

Fix DNN Featurizer sample bug. #3237

merged 1 commit into from
Apr 8, 2019

Conversation

codemzs
Copy link
Member

@codemzs codemzs commented Apr 8, 2019

DNN featurizer sample throws an exception because it cannot find the Resnet ONNX model. This PR fixes it.

fixes #3236

@codemzs codemzs requested review from artidoro and sfilipi April 8, 2019 18:21
@codecov
Copy link

codecov bot commented Apr 8, 2019

Codecov Report

Merging #3237 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3237      +/-   ##
==========================================
- Coverage   72.64%   72.63%   -0.01%     
==========================================
  Files         807      807              
  Lines      145080   145080              
  Branches    16213    16213              
==========================================
- Hits       105391   105379      -12     
- Misses      35269    35279      +10     
- Partials     4420     4422       +2
Flag Coverage Δ
#Debug 72.63% <ø> (-0.01%) ⬇️
#production 68.18% <ø> (-0.01%) ⬇️
#test 88.92% <ø> (-0.01%) ⬇️
Impacted Files Coverage Δ
...c/Microsoft.ML.FastTree/Utils/ThreadTaskManager.cs 79.48% <0%> (-20.52%) ⬇️
...soft.ML.TestFramework/DataPipe/TestDataPipeBase.cs 73.7% <0%> (-0.34%) ⬇️
...ML.Transforms/Text/StopWordsRemovingTransformer.cs 86.1% <0%> (-0.16%) ⬇️

@codemzs codemzs merged commit 1724da8 into dotnet:master Apr 8, 2019
codemzs added a commit to codemzs/machinelearning that referenced this pull request Apr 11, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Mar 23, 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.

DNN Featurizer sample bug.
3 participants