[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#336330: qa.debian.org: adding links to popcon graphs to popcon display page



Package: qa.debian.org
Severity: wishlist
Tags: patch

Dear QA people,

I'm wondering whether it's desirable to include links to Ian Lynagh's
popcon graphs in the popcon display page, like e.g. adding a link to
<http://people.debian.org/~igloo/popcon-graphs/index.php?packages=ascii>
to
<http://qa.debian.org/developer.php?popcon=ascii>.

This for example allows to roughly estimate the user distribution over
the different flavors of Debian. However, for other packages (like e.g.
"kde") the advantage would be nearly non-existing, and I don't know
whether Ian considers his scripts ready for such a use and where the
source is (thus X-Debbugs-CCing him)...

Personally I'd like to have such a link as outlined in the attached
patch against data/ddpo/extract_popcon.py, but I'd also like to hear
other opinions / comments.

Cheers,
Flo
--- extract_popcon.py.orig	2005-10-29 16:51:36.000000000 +0200
+++ extract_popcon.py	2005-10-29 16:51:46.000000000 +0200
@@ -168,12 +168,13 @@
             recent_data = float(result_hash[source][binary][3]['value']) / float(max_data['recent']) * 100
             nofiles_data = float(result_hash[source][binary][4]['value']) / float(max_data['nofiles']) * 100
             print >> popcon, """<tr align="center"%s>""" % color
-            print >> popcon, """<td><small><b>%s</b></small></td>
+            print >> popcon, """<td><small><b>%s</b><br>
+<a href="http://people.debian.org/~igloo/popcon-graphs/index.php?packages=%s";>[graphs]</a></small></td>
 <td><small>%s</small></td><td><small>%.2f &#037;</small></td><td><small>%s</small></td>
 <td><small>%s</small></td><td><small>%.2f &#037;</small></td><td><small>%s</small></td>
 <td><small>%s</small></td><td><small>%.2f &#037;</small></td><td><small>%s</small></td>
 <td><small>%s</small></td><td><small>%.2f &#037;</small></td><td><small>%s</small></td>
-<td><small>%s</small></td><td><small>%.2f &#037;</small></td><td><small>%s</small></td>""" % (binary, 
+<td><small>%s</small></td><td><small>%.2f &#037;</small></td><td><small>%s</small></td>""" % (binary, binary,
                           result_hash[source][binary][0]['value'], inst_data, rank_inst[binary], 
                           result_hash[source][binary][1]['value'], vote_data, rank_vote[binary],
                           result_hash[source][binary][2]['value'], old_data, rank_old[binary],

Attachment: signature.asc
Description: Digital signature


Reply to: