Closed VincentRouvreau closed 3 weeks ago
ok, though I would appreciate a comment on #1056 (comment) (the values may be fine, but using different default values than sklearn deserves in my opinion a quick justification, for instance as a comment in the source code).
57563ef I put the comment in the doc. The rendering (some sphinx dependencies were updated):
Fix #1031 Sorry @martinroyer I was not able to keep the history while taking your changes from #1017 but merging #1017 will still be possible in the future.