-
-
Notifications
You must be signed in to change notification settings - Fork 256
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
Automatically generate the intra doc links #246
Conversation
I think Also, do you mind setting |
Codecov ReportBase: 39.22% // Head: 55.42% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #246 +/- ##
===========================================
+ Coverage 39.22% 55.42% +16.19%
===========================================
Files 92 96 +4
Lines 6085 8936 +2851
===========================================
+ Hits 2387 4953 +2566
- Misses 3698 3983 +285
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Instead of adding I also added the
|
Addresses #245
The doc links are changed from
[comment](struct.StructName.html)
to[comment](StructName)
.I realized that the documentation for TweedieRegressor was not generating. To fix it and to match the public API of
linfa-linear
with the other crates (e.g.KMeans
andKMeansParams
from linfa_clustering), I made the following changes:TweedieRegressorValidParams
(which is not public), is moved forTweedieRegressor
TweedieRegressorParams
is now publicSmall fixes:
FittedCountVectorizer
->CountVectorizer
in linfa-preprocessing