<div dir="ltr">Hi Rain,<div><br></div><div>I would suggest that you start by documenting what your code is meant to do (the structure of the Korjus et al paper makes it pretty difficult to even determine what this technique is, for you to then not to describe it in your own words in your repository), testing it with diverse inputs and ensuring that it is correct. At a glance I can see at least two sources of bugs, and some API design choices which I think could be improved.</div><div><br></div><div>Cheers,</div><div><br></div><div>Joel</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 5 June 2017 at 07:04, Rain Vagel <span dir="ltr"><<a href="mailto:rain.vagel@gmail.com" target="_blank">rain.vagel@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hey,<div><br></div><div>I am a bachelor’s student and for my thesis I implemented a cross-testing function in a scikit-learn compatible way and published it on Github. The paper on which I based my own thesis can be found here: <a href="http://journals.plos.org/plosone/article?id=10.1371/journal.pone.0161788" target="_blank">http://journals.plos.<wbr>org/plosone/article?id=10.<wbr>1371/journal.pone.0161788</a>.</div><div><br></div><div>My project can be found here: <a href="https://github.com/RainVagel/cross-val-cross-test" target="_blank">https://github.com/RainVagel/<wbr>cross-val-cross-test</a>.</div><div><br></div><div>Our original plan was to try and get the algorithm into scikit-learn, but it doesn’t meet the requirements yet. So instead we thought about maybe having it listed in the “Related Projects” page. Is it possible for somebody to take a look and give any feedback?</div><div><br><div>
<div>Sincerely,</div>Rain<br><br><br>

</div>
<br></div></div><br>______________________________<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>
<br></blockquote></div><br></div>