Conversation
|
This will also fix all the blockers for #7165 except the async test one. I'll get a pr up for that one. |
|
Actually it was such a minor change I just added it to this PR as well. |
| ## About | ||
|
|
||
| ML.NET is a cross-platform open-source machine learning framework which makes machine learning accessible to .NET developers. | ||
|
|
There was a problem hiding this comment.
Usually the read me list the major types and small specific sample in the package readme. Are we using the same readme template used in the runtime repo? .
There was a problem hiding this comment.
So it looks like FastTree is the only one that I missed for the major types. I'll get those in. As far as samples go, we have a link to the code samples repo. Do you think thats enough or do you think we should actually put some samples in here? I basically just copied what had been done for the main ML.NET package to these others ones.
tarekgh
left a comment
tarekgh
left a comment
There was a problem hiding this comment.
Left comment/question. LGTM otherwise.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7200 +/- ##
==========================================
- Coverage 69.06% 68.81% -0.26%
==========================================
Files 1401 1267 -134
Lines 268516 259675 -8841
Branches 27812 26933 -879
==========================================
- Hits 185451 178689 -6762
+ Misses 75902 74103 -1799
+ Partials 7163 6883 -280
Flags with carried forward coverage won't be shown. Click here to find out more.
|
3 participants
Update to use preview 5 for building.
Fixed some minor formatting issues that came from the preview 5 update.
Added package readmes to 5 more packages for now, #7195