https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Bug 37463 depends on bug 58428, which changed state.
Bug 58428 Summary: Gerrit metrics: details about review queues
https://bugzilla.wikimedia.org/show_bug.cgi?id=58428
What|Removed |Added
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #43 from Quim Gil q...@wikimedia.org ---
Finally, I have started providing better strings in the form of pull requests:
https://github.com/Bitergia/mediawiki-dashboard/pulls
In the meantime
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Bug 37463 depends on bug 58424, which changed state.
Bug 58424 Summary: Gerrit metrics: average time to review by affiliation
https://bugzilla.wikimedia.org/show_bug.cgi?id=58424
What|Removed |Added
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Bug 37463 depends on bug 58426, which changed state.
Bug 58426 Summary: Gerrit metrics: average time to review by repository
https://bugzilla.wikimedia.org/show_bug.cgi?id=58426
What|Removed |Added
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Quim Gil q...@wikimedia.org changed:
What|Removed |Added
Assignee|a...@bitergia.com|q...@wikimedia.org
---
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #41 from Quim Gil q...@wikimedia.org ---
What remains to close this report:
(In reply to Quim Gil from comment #22)
* the first graph at Are Wikimedia's staff... can be removed; it is a bit
confusing and the second one explains
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #33 from Alvaro a...@bitergia.com ---
Data updated removing l10n and adding Speedy reviews:
http://korma.wmflabs.org/browser/gerrit_review_queue.html
Also the pending metric has being improved.
--
You are receiving this mail
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #34 from Quim Gil q...@wikimedia.org ---
Thank you! Please try to address Comment #22 and the paradox of Babel :)
explained in Comment #24 before our meeting tomorrow. If we want to have all
KPIs solved this month, we need to close
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #35 from Alvaro a...@bitergia.com ---
(In reply to Quim Gil from comment #34)
Thank you! Please try to address Comment #22 and the paradox of Babel :)
explained in Comment #24 before our meeting tomorrow. If we want to have all
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #36 from Quim Gil q...@wikimedia.org ---
Ah, ok, I guess I had confused that -2 with rejected/abandoned. So these -2
are still open. Understood, thank you.
What I don't understand is the flat line that repositories like Babel have.
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #37 from Alvaro a...@bitergia.com ---
(In reply to Quim Gil from comment #36)
Ah, ok, I guess I had confused that -2 with rejected/abandoned. So these
-2 are still open. Understood, thank you.
Ok!
What I don't understand is
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #38 from Quim Gil q...@wikimedia.org ---
(In reply to Alvaro from comment #37)
Remember that the review time is computed for all submissions using the NOW
date.
That changeset was uploaded on May 7, 2013, and if this was
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #39 from Alvaro a...@bitergia.com ---
Quim, metric logic fixed following the above definition.
You have the new data and viz in korma:
http://korma.wmflabs.org/browser/gerrit_review_queue.html
Ups, I am sorry about the first
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #40 from Alvaro a...@bitergia.com ---
(In reply to Alvaro from comment #39)
Quim, metric logic fixed following the above definition.
You have the new data and viz in korma:
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #31 from Nemo federicol...@tiscali.it ---
(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_bychanges.changed_by
Can't review the
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #32 from Quim Gil q...@wikimedia.org ---
(In reply to Nemo from comment #31)
(In reply to Alvaro from comment #30)
min_days_for_review = 0.042 # one hour
This I don't like. Speedy reviews are still reviews
Agreed, speedy
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #19 from Alvaro a...@bitergia.com ---
Update:
The list of repos is now sorted by the median age of the changes opened at a
given time (month) and the evol in time of this metric is shown.
Also, for affiliations this metric has
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #20 from Alvaro a...@bitergia.com ---
About to calculate the age based on the date of the last patchset uploaded
analyzing bicho gerrit database, the upload event is not stored, but the
actions against patchsets (Code-Review,
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #21 from Alvaro a...@bitergia.com ---
(In reply to Alvaro from comment #20)
About to calculate the age based on the date of the last patchset uploaded
analyzing bicho gerrit database, the upload event is not stored, but the
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #22 from Quim Gil q...@wikimedia.org ---
VERY COOL! These news graphs are very useful.
Calculating last code review is not bad, and in fact it a useful measure per
se. Jenkins reviews every new patchset within seconds, so at least
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #23 from Nemo federicol...@tiscali.it ---
I note that all graphs are still completely meaningless because they still
don't exclude self merges.
Babel
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #24 from Quim Gil q...@wikimedia.org ---
(In reply to Nemo from comment #23)
I note that all graphs are still completely meaningless because they still
don't exclude self merges.
Could you point to a Gerrit changeset with an
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #25 from Jesus M. Gonzalez-Barahona j...@bitergia.com ---
(In reply to Quim Gil from comment #24)
At the very least, l10n-bot should have its own line in graphs, it doesn't
make sense to mix it with Independent or Unknown
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #26 from Jesus M. Gonzalez-Barahona j...@bitergia.com ---
(In reply to Quim Gil from comment #24)
(In reply to Nemo from comment #23)
I note that all graphs are still completely meaningless because they still
don't exclude self
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #27 from Nemo federicol...@tiscali.it ---
(In reply to Jesus M. Gonzalez-Barahona from comment #26)
Nemo, are you referring to code review processes where the uploader of the
changeset and the person accepting the merge is same
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #28 from Quim Gil q...@wikimedia.org ---
Ok, I see:
https://gerrit.wikimedia.org/r/#/c/114180/
OwnerMark Bergsma
UploadedFeb 19, 2014 10:48 AM
Mark BergsmaPatch Set 1: Code-Review+2Feb 19 10:50 AM
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #29 from Jesus M. Gonzalez-Barahona j...@bitergia.com ---
Thanks to both of you. Definitely, we will look into that to fix our query.
--
You are receiving this mail because:
You are on the CC list for the bug.
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #30 from Alvaro a...@bitergia.com ---
Hi guys, about the automerge and instant reviews we are already dealing with
this in the last metrics:
https://github.com/VizGrimoire/VizGrimoireR/blob/mediawiki/vizgrimoire/SCR.py#L728
This
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #18 from Quim Gil q...@wikimedia.org ---
We have a new graph at
http://korma.wmflabs.org/browser/gerrit_review_queue.html labeled
How long does it take to review code contributions? For pending reviews
(final labels pending)
It
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #17 from Quim Gil q...@wikimedia.org ---
Yes, labels and descriptions will be reviewed before closing this task. The ?
icons next to the tables contain some information.
Improvements to the text of the page can be proposed directly
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #16 from Nemo federicol...@tiscali.it ---
The graph would use a set of legends or reference manual. For instance, I have
no idea at all what Reviews submitted and Reviews new could mean.
--
You are receiving this mail because:
You
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #15 from Quim Gil q...@wikimedia.org ---
Status update (Nemo, I will leave your comment to Alvaro since I don't know
better)
We have some graphs here:
http://korma.wmflabs.org/browser/gerrit_review_queue.html
How long is the
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Bug 37463 depends on bug 58349, which changed state.
Bug 58349 Summary: Graph of pending reviews in Gerrit shows wrong data
https://bugzilla.wikimedia.org/show_bug.cgi?id=58349
What|Removed |Added
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Nemo federicol...@tiscali.it changed:
What|Removed |Added
CC||pleasest...@live.com
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #13 from Quim Gil q...@wikimedia.org ---
As per bug 53485 comment 35 and 39, let's not count self-merges (reviews merged
by the author of the patch) in any average time to review metrics.
--
You are receiving this mail because:
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Quim Gil q...@wikimedia.org changed:
What|Removed |Added
Depends on||58424
--
You are
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Quim Gil q...@wikimedia.org changed:
What|Removed |Added
Depends on||58426
--
You are
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Quim Gil q...@wikimedia.org changed:
What|Removed |Added
Depends on||58428
--
You are
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #12 from Quim Gil q...@wikimedia.org ---
Summary of a chat with Alvaro reviewing
https://www.mediawiki.org/wiki/Community_metrics#Gerrit_review_queue :
* How long is the Gerrit review queue over time?
Answered an the Pending
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Quim Gil q...@wikimedia.org changed:
What|Removed |Added
Depends on||58349
--
You are
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #11 from Alvaro a...@bitergia.com ---
Added info about time to review to the gerrit panel in korma:
http://korma.wmflabs.org/browser/scr.html
The average time for time to review is: 3.7333
And you can see also the evolution in
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #10 from Alvaro a...@bitergia.com ---
Ok, we have some results no. The gerrit backend for Bicho has being improved in
order to get the MERGED and ABANDONED real event dates.
We have added some new vizs to the gerrit panel in korma
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
Quim Gil q...@wikimedia.org changed:
What|Removed |Added
Priority|Normal |High
https://bugzilla.wikimedia.org/show_bug.cgi?id=37463
--- Comment #9 from Alvaro a...@bitergia.com ---
Great! Next week time to start working on it!
--
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
44 matches
Mail list logo