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

[Florian Ernst]

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],

signature.asc
Description: Digital signature