-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Adding vector tests for KeyToValue and ValueToKey #5090
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5090 +/- ##
==========================================
- Coverage 75.66% 75.65% -0.01%
==========================================
Files 993 993
Lines 178168 178170 +2
Branches 19122 19126 +4
==========================================
- Hits 134802 134799 -3
- Misses 38143 38146 +3
- Partials 5223 5225 +2
|
nit: Please add a newline above. But more importantly, ValueToKeyMapping (and related) transforms have caused a bunch of issues in Nimbus. Can you please run the Nimbus suite of tests and make sure that nothing is broken there as a result of these changes? #Resolved Refers to: src/Microsoft.ML.Data/Transforms/ValueToKeyMappingTransformer.cs:813 in ebea3a0. [](commit_id = ebea3a0, deletion_comment = False) |
No description provided.