Confirmed users
513
edits
Auto-tools/Projects/DevelopmentMetrics: Difference between revisions
Auto-tools/Projects/DevelopmentMetrics (view source)
Revision as of 17:08, 31 December 2015
, 31 December 2015add Requires VPN
Klahnakoski (talk | contribs) (column alignment) |
Klahnakoski (talk | contribs) (add Requires VPN) |
||
| Line 21: | Line 21: | ||
{| | {| | ||
|- | |- | ||
| style="vertical-align:top;width:300px;" | [http://people.mozilla.org/~klahnakoski/review/Review-byTop.html Top 30 Reviewers] | | style="vertical-align:top;width:300px;" | [http://people.mozilla.org/~klahnakoski/review/Review-byTop.html Top 30 Reviewers]<br>Requires VPN | ||
| Shows the top reviewers from the past 6 weeks, the number of reviews they did, and the number in their queue right now. The original objective of this dashboard is to confirm some engineers have too many reviews. This dashboard is fine for identifying egregious differences, but poor for individual comparison; it seems personal style, team style, and code under review all impact the size and number of reviews done. | | Shows the top reviewers from the past 6 weeks, the number of reviews they did, and the number in their queue right now. The original objective of this dashboard is to confirm some engineers have too many reviews. This dashboard is fine for identifying egregious differences, but poor for individual comparison; it seems personal style, team style, and code under review all impact the size and number of reviews done. | ||
|- | |- | ||
| style="vertical-align:top;width:300px;" | [http://people.mozilla.org/~klahnakoski/review/ReviewIntensity_First.html Time to First Incoming Review] | | style="vertical-align:top;width:300px;" | [http://people.mozilla.org/~klahnakoski/review/ReviewIntensity_First.html Time to First Incoming Review]<br>Requires VPN | ||
| Counts review requests and review completion rates. Also breaks down aggregate review age. The first review has been shown to take the longest, probably due to people synchronization issues. Subsequent reviews were almost instantaneous, so we exclude them. This dashboard is not good for fine grained comparison of teams because the nature of the bugs, and the process used, is often too different. The response time INCLUDES WEEKENDS. | | Counts review requests and review completion rates. Also breaks down aggregate review age. The first review has been shown to take the longest, probably due to people synchronization issues. Subsequent reviews were almost instantaneous, so we exclude them. This dashboard is not good for fine grained comparison of teams because the nature of the bugs, and the process used, is often too different. The response time INCLUDES WEEKENDS. | ||
|- | |- | ||
| style="vertical-align:top;width:300px;" | [http://people.mozilla.org/~klahnakoski/review/Review-byReviewer.html Incoming Review by Individual] | | style="vertical-align:top;width:300px;" | [http://people.mozilla.org/~klahnakoski/review/Review-byReviewer.html Incoming Review by Individual]<br>Requires VPN | ||
| Top reviewers, but this time with some breakdown by week and component. Interesting, but not actionable. | | Top reviewers, but this time with some breakdown by week and component. Interesting, but not actionable. | ||
|- style="vertical-align:top;" | |- style="vertical-align:top;" | ||
| [http://people.mozilla.org/~klahnakoski/review/ReviewIntensity.html Incoming Reviews] | | [http://people.mozilla.org/~klahnakoski/review/ReviewIntensity.html Incoming Reviews]<br>Requires VPN | ||
| Statistics for all reviews. Subsequent reviews of a bug happen much faster than the first. Maybe the fast review speed is an administrative anomaly; reviews are added after the fact, and marked as reviewed. Maybe, the requester and reviewer are working closely to remove the remaining nits. | | Statistics for all reviews. Subsequent reviews of a bug happen much faster than the first. Maybe the fast review speed is an administrative anomaly; reviews are added after the fact, and marked as reviewed. Maybe, the requester and reviewer are working closely to remove the remaining nits. | ||
|- | |- | ||
| style="vertical-align:top;width:300px;" | [http://people.mozilla.org/~klahnakoski/review/Reviews_Pending_18.html Reviews Requested in the Past 18 weeks] | | style="vertical-align:top;width:300px;" | [http://people.mozilla.org/~klahnakoski/review/Reviews_Pending_18.html Reviews Requested in the Past 18 weeks]<br>Requires VPN | ||
| An attempt to better understand the aggregate response time; while treating "old" reviews (those over 18weeks), as something else entirely. | | An attempt to better understand the aggregate response time; while treating "old" reviews (those over 18weeks), as something else entirely. | ||
|} | |} | ||