From: "Theodore Y. Ts'o" <tytso@mit.edu>
To: Joe Perches <joe@perches.com>
Cc: James Bottomley <James.Bottomley@HansenPartnership.com>,
Tech Board Discuss
<Tech-board-discuss@lists.linux-foundation.org>,
Shuah Khan <shuah@kernel.org>,
"ksummit-discuss@lists.linuxfoundation.org"
<ksummit-discuss@lists.linuxfoundation.org>
Subject: Re: [Tech-board-discuss] [Ksummit-discuss] TAB non-nomination
Date: Sat, 10 Nov 2018 16:21:59 -0500 [thread overview]
Message-ID: <20181110212159.GA12818@thunk.org> (raw)
In-Reply-To: <e0ec26aa685f53e741f66e757a2cf29307ce3b7e.camel@perches.com>
On Fri, Nov 09, 2018 at 11:23:26AM -0800, Joe Perches wrote:
> I believe that did not happen as described as at least
> I was not asked for input/comment/sign-off and I ahm
> well within that top 200 or so list.
I'm not sure precisely which metrics Greg used when he created his
list, but using "git log --since="July 2017" v4.18" and piping it into
the perl script I use to count Signed-off-by, Acked-by, and
Reviewed-by, lines, you're about #240; so you're in the top 250, but
not the top 200.
There are other metrics that could be used as well, such using the
number of lines added/removed to create the ranked list. Different
metrics will have differing amounts of bias for or against people who
generate large number of "rename variables to random new types" or
"whitespace cleanups" commits.
To be honest, that's one of the weaknesses of only using git
statistics. They have the advantage that they are objective; however,
especially if the exact metric which is used is revealed, such systems
can also be easily gamed. (e.g., do you create a single commit that
fixes all of the white space in one subsystem or use a separate commit
for each file?)
And of course, using commit statistics may not accurately measure the
value of say, a set of patches which add support for a new
architecture or fixes a tricky locking problem, versus commits which
only fix checkpatch complaints. So I will be the first to admit that
using git statistics has its limitations.
Regards,
- Ted
next prev parent reply other threads:[~2018-11-10 21:21 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-09 0:04 [Tech-board-discuss] TAB non-nomination James Bottomley
2018-11-09 0:29 ` Steven Rostedt
2018-11-09 3:30 ` [Tech-board-discuss] [Ksummit-discuss] " Chris Mason
2018-11-09 17:52 ` Shuah Khan
2018-11-09 19:03 ` Theodore Y. Ts'o
2018-11-09 19:23 ` Joe Perches
2018-11-10 21:21 ` Theodore Y. Ts'o [this message]
2018-11-10 21:47 ` Joe Perches
2018-11-12 17:15 ` James Morris
2018-11-09 20:17 ` [Tech-board-discuss] better hot-topic discussion processes was: " Jason Cooper
2018-11-10 19:26 ` [Tech-board-discuss] [Ksummit-discuss] " Chris Mason
2018-11-10 21:55 ` Jason Cooper
2018-11-14 18:25 ` [Tech-board-discuss] [Ksummit-discuss] " Geert Uytterhoeven
2018-11-09 19:54 ` Frank Rowand
2018-11-10 19:15 ` Chris Mason
2018-11-10 21:59 ` Jason Cooper
2018-11-11 3:18 ` Frank Rowand
2018-11-11 5:57 ` Theodore Y. Ts'o
2018-11-12 4:44 ` NeilBrown
2018-11-12 4:54 ` NeilBrown
2018-11-12 17:00 ` Steven Rostedt
2018-11-13 16:49 ` Jani Nikula
2018-11-13 19:59 ` Laurent Pinchart
2018-11-14 17:28 ` Mark Brown
2018-11-09 17:19 ` Stephen Hemminger
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181110212159.GA12818@thunk.org \
--to=tytso@mit.edu \
--cc=James.Bottomley@HansenPartnership.com \
--cc=Tech-board-discuss@lists.linux-foundation.org \
--cc=joe@perches.com \
--cc=ksummit-discuss@lists.linuxfoundation.org \
--cc=shuah@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox