[tor-commits] [metrics-tasks/master] Stop after removing last but one relay (#1854).

karsten at torproject.org karsten at torproject.org
Tue Oct 23 16:32:57 UTC 2012

commit a9ebf2d03a39f4c72453bdf3e2eae7a27988b021
Author: Karsten Loesing <karsten.loesing at gmx.net>
Date:   Tue Oct 23 12:27:27 2012 -0400

    Stop after removing last but one relay (#1854).
    Calculating the maximum probability difference between pristine and empty
    consensus doesn't make much sense.  More importantly, the current
    implementation violates the function relationship between linf and
    min_adv_bw, because we're not increasing min_adv_bw in the last run and
    thereby outputting two different linfs for the same min_adv_bw.  Should be
    safe to omit that final run, because we're not looking for the case where
    we remove all relays from the consensus anyway.  I hope.
 task-1854/pylinf.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/task-1854/pylinf.py b/task-1854/pylinf.py
index 3bc6de1..1a6e2d2 100644
--- a/task-1854/pylinf.py
+++ b/task-1854/pylinf.py
@@ -121,7 +121,7 @@ def run(file_name):
     omitted_routers = 0
     min_adv_bw = routers[0].advertised_bw
-    while(omitted_routers<=len(routers)):
+    while(omitted_routers<len(routers)):
         total_bw = 0
         # this is the difference btw probability of choosing a relay in pristine

More information about the tor-commits mailing list