--- Comment #31 from Nemo <> ---
(In reply to Alvaro from comment #30)
> This is the basic query for getting the time to review and we are filtering:
>  filters += " AND i.submitted_by<>changes.changed_by "

Can't review the query syntax but the principle is ok.

> and also
>  min_days_for_review = 0.042 # one hour

This I don't like. Speedy reviews are still reviews: sure, they may be mostly
typo fixes or changes coming from pair programming or whatever, but it's still
valid. Don't draw an arbitrary line, instead use appropriate percentiles. If
the median is still too skewed, use the 75th percentile as the WMF is doing...
I don't remember where.

> We will also filter out "i18n" submissions.

This is ok. Ideally you'd leave them in *one* graph i.e. the page on l10n-bot
account itself, so that we have one place where to monitor its activity. But
it's not necessary, I can also get such data from ohloh once I convince
Nikerabbit to reply to its emails. :P

You are receiving this mail because:
You are on the CC list for the bug.
Wikibugs-l mailing list

Reply via email to