<div dir="ltr"><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif;font-size:large;color:rgb(51,51,51)">I've added this PR, and I addressed in the comments some of your concerns (publications, comparison to affinity propagation, etc).</div><div class="gmail_default" style="font-family:"trebuchet ms",sans-serif;font-size:large;color:rgb(51,51,51)"><br></div><div class="gmail_default"><font color="#333333" face="trebuchet ms, sans-serif" size="4"><a href="https://github.com/scikit-learn/scikit-learn/pull/9329">https://github.com/scikit-learn/scikit-learn/pull/9329</a></font><br></div><div class="gmail_default"><font color="#333333" face="trebuchet ms, sans-serif" size="4"><br></font></div><div class="gmail_default"><font color="#333333" face="trebuchet ms, sans-serif" size="4">I'd love for you to review, since this is my first PR in the scikit learn repository</font></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 12, 2017 at 12:04 AM, Olivier Grisel <span dir="ltr"><<a href="mailto:olivier.grisel@ensta.org" target="_blank">olivier.grisel@ensta.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If this is the first time you contribute, please make sure to<br>
carefully read the contributors guide till the end:<br>
<br>
<a href="http://scikit-learn.org/stable/developers/contributing.html" rel="noreferrer" target="_blank">http://scikit-learn.org/<wbr>stable/developers/<wbr>contributing.html</a><br>
<br>
In particular, make sure to follow the estimators API conventions for<br>
your PR to get a chance to be reviewed. In particular the gist you<br>
linked to is not compatible with the scikit-learn estimators API.<br>
<br>
Personally I have never heard of Markov clustering, so it's hard for<br>
me to assess whether it should be included in the project or not. It<br>
would really help if you could demonstrate its performance on a<br>
publicly available dataset where is does significantly better than all<br>
the other clustering algorithms already implemented in scikit-learn<br>
(both in terms of training speed and in terms of cluster quality /<br>
stability, although this latter point is very domain dependent).<br>
<br>
As a side note, if this is the first time you contribute to the<br>
project, it's probably best to have a look at how other pull requests<br>
are being reviewed (by reading the comment threads of other PRs) and<br>
maybe start by a small pull request to fix small bug (with a<br>
non-regression test) or tackle some documentation issues. Adding new<br>
estimators takes a lot of effort to review (we need tests, docs,<br>
updated examples) and assume some familiarity with the existing code<br>
base.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Olivier<br>
</font></span><div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
scikit-learn mailing list<br>
<a href="mailto:scikit-learn@python.org">scikit-learn@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/scikit-learn" rel="noreferrer" target="_blank">https://mail.python.org/<wbr>mailman/listinfo/scikit-learn</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div><span style="color:rgb(102,102,102)"><font size="2"><span style="font-family:verdana,sans-serif"><i>Uri Goren,<br>Software innovator<br></i></span></font></span></div><span style="color:rgb(102,102,102)"><font size="2"><span style="font-family:verdana,sans-serif"><i>Phone: +972-507-649-650<br></i></span></font></span></div><div><span style="color:rgb(102,102,102)"><font size="2"><span style="font-family:verdana,sans-serif"><i>EMail: <a href="mailto:uri@goren4u.com" target="_blank">uri@goren4u.com</a><br></i></span></font></span></div><span style="color:rgb(102,102,102)"><font size="2"><span style="font-family:verdana,sans-serif"><i>Linkedin: <a href="http://il.linkedin.com/in/ugoren/" title="View public profile" name="UNIQUE_ID_SafeHtmlFilter_webProfileURL" target="_blank">il.linkedin.com/in/ugoren/</a></i></span></font></span></div></div>
</div>