DOC: rephrase the role of core devs #28912
Conversation
Co-authored-by: Jérémie du Boisberranger <[email protected]>
doc/about.rst
Outdated
| roles, however a complete list of contributors can be found `on github | ||
| <https://github.com/scikit-learn/scikit-learn/graphs/contributors>`__. |
There was a problem hiding this comment.
that page only shows top 100 people, which at this point means minimum 24 PRs merged.
There was a problem hiding this comment.
I rephrased slightly to be more nuanced
Co-authored-by: Adrin Jalali <[email protected]>
Co-authored-by: Adrin Jalali <[email protected]>
Co-authored-by: Adrin Jalali <[email protected]>
Co-authored-by: Adrin Jalali <[email protected]>
Co-authored-by: Jérémie du Boisberranger <[email protected]>
… all teams are core contributors
Co-authored-by: Jérémie du Boisberranger <[email protected]>
Co-authored-by: Jérémie du Boisberranger <[email protected]>
Co-authored-by: Jérémie du Boisberranger <[email protected]>
Co-authored-by: Jérémie du Boisberranger <[email protected]>
jjerphan
left a comment
jjerphan
left a comment
There was a problem hiding this comment.
Thank you, @GaelVaroquaux.
LGTM modulo a few nitpicky comments, questions, and a suggestion regarding one element of yesterday's discussion (i.e the rewording of "Authors").
|
|
||
| Authors | ||
| ------- | ||
| .. _authors: |
There was a problem hiding this comment.
Should this tag be adapted? Shall we also credit the original academical authors of scikit-learn and its associated publication?
There was a problem hiding this comment.
We can adapt this tag if you think that it is important. I did not think it was, as it's purely an internal tag, and I was worried of doing my git grep wrong and thus missing some places pointing there.
I'm not in favor of crediting the original academic authors and the associated publication more than they already are: this list reflect the people involved in the first couple of years, which is a small fraction of the total road that got scikit-learn to where it is today.
There was a problem hiding this comment.
Actually, regarding the tags, I added it to make sure that old html links that point to the anchor "#authors" in html still worked.
I'm relearning an old lesson: each time one does a non obvious thing, it should come with a comment :)
doc/about.rst
Outdated
| The following people are currently maintainers, in charge of | ||
| consolidating scikit-learn's development and maintenance: | ||
|
|
||
| .. include:: authors.rst |
There was a problem hiding this comment.
Do we also want to rename authors.rst?
There was a problem hiding this comment.
Done!
I hope that I did not screw something up here. I guess that we'll know next time we update the contributor list, eg when updating the emeritus
4 participants
A slight adjustment in phrasing and framing to give more visibility to all contributors, and to avoid suggesting that core devs are the only authors.
This PR is done to implement the discussion of today's meeting.