Commit fd74a4c4 authored by juga's avatar juga Committed by Matt Traudt
Browse files

Stop calculating median on empty list

parent d3bf951f
...@@ -320,13 +320,13 @@ class V3BWLine(object): ...@@ -320,13 +320,13 @@ class V3BWLine(object):
# Note how this isn't some measured-by-us average of bandwidth. It's # Note how this isn't some measured-by-us average of bandwidth. It's
# the first value on the 'bandwidth' line in the relay's server # the first value on the 'bandwidth' line in the relay's server
# descriptor. # descriptor.
relay_average_bw = median([ bw = median_bw
r.relay_average_bandwidth for r in results relay_average_bw = [r.relay_average_bandwidth for r in results
if r.relay_average_bandwidth is not None]) if r.relay_average_bandwidth is not None]
if median_bw > relay_average_bw: if relay_average_bw:
bw = relay_average_bw median_relay_average_bw = median(relay_average_bw)
else: if median_bw > median_relay_average_bw:
bw = median_bw bw = median_relay_average_bw
# convert to KB and ensure it's at least 1 # convert to KB and ensure it's at least 1
bw_kb = max(round(bw / 1024), 1) bw_kb = max(round(bw / 1024), 1)
return bw_kb return bw_kb
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment