dotnet / machinelearning

ML.NET is an open source and cross-platform machine learning framework for .NET.
https://dot.net/ml
MIT License
9.05k stars 1.88k forks source link

Add the components governance file `cgmanifest.json` for tokenizer's vocab files #7283

Closed tarekgh closed 2 weeks ago

tarekgh commented 3 weeks ago

CC @GrabYourPitchforks

tarekgh commented 3 weeks ago

@ericstj I added the component governance file cgmanifest.json in the root of the repo. We can store it anywhere in the repo if we want to. Please let me know if you are ok with that.

CC @michaelgsharp

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 68.86%. Comparing base (a9b4212) to head (c2171cd). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7283 +/- ## ========================================== - Coverage 68.87% 68.86% -0.01% ========================================== Files 1467 1467 Lines 273955 273954 -1 Branches 28380 28380 ========================================== - Hits 188697 188671 -26 - Misses 77946 77973 +27 + Partials 7312 7310 -2 ``` | [Flag](https://app.codecov.io/gh/dotnet/machinelearning/pull/7283/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | Coverage Δ | | |---|---|---| | [Debug](https://app.codecov.io/gh/dotnet/machinelearning/pull/7283/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | `68.86% <100.00%> (-0.01%)` | :arrow_down: | | [production](https://app.codecov.io/gh/dotnet/machinelearning/pull/7283/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | `63.32% <ø> (-0.02%)` | :arrow_down: | | [test](https://app.codecov.io/gh/dotnet/machinelearning/pull/7283/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | `89.18% <100.00%> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files with missing lines](https://app.codecov.io/gh/dotnet/machinelearning/pull/7283?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet) | Coverage Δ | | |---|---|---| | [...Microsoft.ML.Tokenizers/Model/TiktokenTokenizer.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/7283?src=pr&el=tree&filepath=src%2FMicrosoft.ML.Tokenizers%2FModel%2FTiktokenTokenizer.cs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-c3JjL01pY3Jvc29mdC5NTC5Ub2tlbml6ZXJzL01vZGVsL1Rpa3Rva2VuVG9rZW5pemVyLmNz) | `78.28% <ø> (ø)` | | | [...oft.ML.Tokenizers.Data.Tests/TokenizerDataTests.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/7283?src=pr&el=tree&filepath=test%2FMicrosoft.ML.Tokenizers.Data.Tests%2FTokenizerDataTests.cs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-dGVzdC9NaWNyb3NvZnQuTUwuVG9rZW5pemVycy5EYXRhLlRlc3RzL1Rva2VuaXplckRhdGFUZXN0cy5jcw==) | `100.00% <ø> (ø)` | | | [...est/Microsoft.ML.Tokenizers.Tests/TiktokenTests.cs](https://app.codecov.io/gh/dotnet/machinelearning/pull/7283?src=pr&el=tree&filepath=test%2FMicrosoft.ML.Tokenizers.Tests%2FTiktokenTests.cs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet#diff-dGVzdC9NaWNyb3NvZnQuTUwuVG9rZW5pemVycy5UZXN0cy9UaWt0b2tlblRlc3RzLmNz) | `98.98% <100.00%> (ø)` | | ... and [9 files with indirect coverage changes](https://app.codecov.io/gh/dotnet/machinelearning/pull/7283/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dotnet)
tarekgh commented 2 weeks ago

/ba-g the failed test is recognized by the build analysis but somehow the build analysis still complaining that helix returned failure.