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

Added onnx export support for KeyToValueMappingTransformer #4455

wants to merge 1 commit into
base: master


Copy link

harishsk commented Nov 7, 2019

No description provided.

@harishsk harishsk requested a review from dotnet/mlnet-core as a code owner Nov 7, 2019
@harishsk harishsk requested review from ganik, codemzs and eerhardt Nov 7, 2019
ganik approved these changes Nov 13, 2019
Copy link

ganik left a comment


@harishsk harishsk force-pushed the harishsk:keytovalue2 branch from eb2c80b to fb68142 Nov 13, 2019
@harishsk harishsk force-pushed the harishsk:keytovalue2 branch from fb68142 to 1a9b58c Nov 14, 2019

This comment has been minimized.

Copy link

codecov bot commented Nov 14, 2019

Codecov Report

❗️ No coverage uploaded for pull request base (master@c1e190a). Click here to learn what that means.
The diff coverage is 78.88%.

@@            Coverage Diff            @@
##             master    #4455   +/-   ##
  Coverage          ?   74.86%           
  Files             ?      908           
  Lines             ?   159862           
  Branches          ?    17214           
  Hits              ?   119679           
  Misses            ?    35371           
  Partials          ?     4812
Flag Coverage Δ
#Debug 74.86% <78.88%> (?)
#production 70.22% <60.46%> (?)
#test 90.19% <95.74%> (?)
Impacted Files Coverage Δ
src/Microsoft.ML.Data/Transforms/KeyToValue.cs 76.9% <60.46%> (ø)
test/Microsoft.ML.Tests/OnnxConversionTest.cs 95.1% <95.74%> (ø)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
2 participants
You can’t perform that action at this time.