Skip to content

Activate OnnxTransform unit tests for MacOS #2695

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

Merged
merged 4 commits into from
Mar 20, 2019

Conversation

jignparm
Copy link
Contributor

OnnxRuntime now supports MacOS runtimes. Activate OnnxTransform unit tests for MacOS.

@codecov
Copy link

codecov bot commented Feb 22, 2019

Codecov Report

Merging #2695 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2695      +/-   ##
==========================================
+ Coverage   72.41%   72.41%   +<.01%     
==========================================
  Files         803      803              
  Lines      143851   143851              
  Branches    16173    16173              
==========================================
+ Hits       104171   104173       +2     
+ Misses      35258    35257       -1     
+ Partials     4422     4421       -1
Flag Coverage Δ
#Debug 72.41% <0%> (ø) ⬆️
#production 68.09% <ø> (ø) ⬆️
#test 88.61% <0%> (ø) ⬆️
Impacted Files Coverage Δ
...t.ML.TestFramework/Attributes/OnnxFactAttribute.cs 83.33% <0%> (ø) ⬆️
...ML.TestFramework/Attributes/OnnxTheoryAttribute.cs 83.33% <0%> (ø) ⬆️
...ML.Transforms/Text/StopWordsRemovingTransformer.cs 85.78% <0%> (+0.15%) ⬆️
...soft.ML.Data/DataLoadSave/Text/TextLoaderCursor.cs 84.9% <0%> (+0.2%) ⬆️

@jignparm jignparm changed the title WIP - Update OnnxTransform unit tests for MacOS CI leg Activate OnnxTransform unit tests for MacOS Mar 15, 2019
Copy link
Contributor

@Anipik Anipik left a comment

Choose a reason for hiding this comment

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

LGTM

@shauheen shauheen requested a review from wschin March 18, 2019 21:43
@jignparm jignparm merged commit ce56462 into dotnet:master Mar 20, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Mar 24, 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.

3 participants