Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(380)

Unified Diff: webrtc/modules/remote_bitrate_estimator/test/plot_dynamics.py

Issue 2812273002: Fix lint errors to enable stricter PyLint rules (Closed)
Patch Set: Rebased Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/remote_bitrate_estimator/test/plot_dynamics.py
diff --git a/webrtc/modules/remote_bitrate_estimator/test/plot_dynamics.py b/webrtc/modules/remote_bitrate_estimator/test/plot_dynamics.py
index 40f17785983f5751522b560d640bd3dc9d52fa3e..51ef06ef0af09254632328928e4e1a39ec6b563b 100755
--- a/webrtc/modules/remote_bitrate_estimator/test/plot_dynamics.py
+++ b/webrtc/modules/remote_bitrate_estimator/test/plot_dynamics.py
@@ -22,7 +22,7 @@ import re
import sys
# Change this to True to save the figure to a file. Look below for details.
-save_figure = False
+SAVE_FIGURE = False
class ParsePlotLineException(Exception):
def __init__(self, reason, line):
@@ -31,7 +31,7 @@ class ParsePlotLineException(Exception):
self.line = line
-def parse_plot_line(line):
+def ParsePlotLine(line):
split_line = line.split()
if len(split_line) != 5:
raise ParsePlotLineException("Expected 5 arguments on line", line)
@@ -51,7 +51,7 @@ def parse_plot_line(line):
return (var_name, ssrc, alg_name, time, value)
-def generate_label(var_name, ssrc, ssrc_count, alg_name):
+def GenerateLabel(var_name, ssrc, ssrc_count, alg_name):
label = var_name
if ssrc_count > 1 or ssrc != "0":
label = label + " flow " + ssrc
@@ -65,18 +65,18 @@ class Figure(object):
self.name = name
self.subplots = []
- def addSubplot(self, var_names, xlabel, ylabel):
+ def AddSubplot(self, var_names, xlabel, ylabel):
self.subplots.append(Subplot(var_names, xlabel, ylabel))
- def addSample(self, var_name, ssrc, alg_name, time, value):
+ def AddSample(self, var_name, ssrc, alg_name, time, value):
for s in self.subplots:
- s.addSample(var_name, ssrc, alg_name, time, value)
+ s.AddSample(var_name, ssrc, alg_name, time, value)
- def plotFigure(self, fig):
+ def PlotFigure(self, fig):
n = len(self.subplots)
for i in range(n):
- ax = fig.add_subplot(n, 1, i+1)
- self.subplots[i].plotSubplot(ax)
+ axis = fig.add_subplot(n, 1, i+1)
+ self.subplots[i].PlotSubplot(axis)
class Subplot(object):
@@ -86,7 +86,7 @@ class Subplot(object):
self.var_names = var_names
self.samples = dict()
- def addSample(self, var_name, ssrc, alg_name, time, value):
+ def AddSample(self, var_name, ssrc, alg_name, time, value):
if var_name not in self.var_names:
return
@@ -99,9 +99,9 @@ class Subplot(object):
self.samples[alg_name][ssrc][var_name].append((time, value))
- def plotSubplot(self, ax):
- ax.set_xlabel(self.xlabel)
- ax.set_ylabel(self.ylabel)
+ def PlotSubplot(self, axis):
+ axis.set_xlabel(self.xlabel)
+ axis.set_ylabel(self.ylabel)
count = 0
for alg_name in self.samples.keys():
@@ -113,7 +113,7 @@ class Subplot(object):
y = numpy.array(y)
ssrc_count = len(self.samples[alg_name].keys())
- l = generate_label(var_name, ssrc, ssrc_count, alg_name)
+ l = GenerateLabel(var_name, ssrc, ssrc_count, alg_name)
plt.plot(x, y, label=l, linewidth=2.0)
count += 1
@@ -124,29 +124,29 @@ class Subplot(object):
def main():
receiver = Figure("PacketReceiver")
- receiver.addSubplot(['Throughput_kbps', 'MaxThroughput_', 'Capacity_kbps',
+ receiver.AddSubplot(['Throughput_kbps', 'MaxThroughput_', 'Capacity_kbps',
'PerFlowCapacity_kbps', 'MetricRecorderThroughput_kbps'],
"Time (s)", "Throughput (kbps)")
- receiver.addSubplot(['Delay_ms_', 'Delay_ms'], "Time (s)",
+ receiver.AddSubplot(['Delay_ms_', 'Delay_ms'], "Time (s)",
"One-way delay (ms)")
- receiver.addSubplot(['Packet_Loss_'], "Time (s)", "Packet Loss Ratio")
+ receiver.AddSubplot(['Packet_Loss_'], "Time (s)", "Packet Loss Ratio")
kalman_state = Figure("KalmanState")
- kalman_state.addSubplot(['kc', 'km'], "Time (s)", "Kalman gain")
- kalman_state.addSubplot(['slope_1/bps'], "Time (s)", "Slope")
- kalman_state.addSubplot(['var_noise'], "Time (s)", "Var noise")
+ kalman_state.AddSubplot(['kc', 'km'], "Time (s)", "Kalman gain")
+ kalman_state.AddSubplot(['slope_1/bps'], "Time (s)", "Slope")
+ kalman_state.AddSubplot(['var_noise'], "Time (s)", "Var noise")
detector_state = Figure("DetectorState")
- detector_state.addSubplot(['offset_ms'], "Time (s)", "Offset")
- detector_state.addSubplot(['gamma_ms'], "Time (s)", "Gamma")
+ detector_state.AddSubplot(['offset_ms'], "Time (s)", "Offset")
+ detector_state.AddSubplot(['gamma_ms'], "Time (s)", "Gamma")
trendline_state = Figure("TrendlineState")
- trendline_state.addSubplot(["accumulated_delay_ms", "smoothed_delay_ms"],
+ trendline_state.AddSubplot(["accumulated_delay_ms", "smoothed_delay_ms"],
"Time (s)", "Delay (ms)")
- trendline_state.addSubplot(["trendline_slope"], "Time (s)", "Slope")
+ trendline_state.AddSubplot(["trendline_slope"], "Time (s)", "Slope")
target_bitrate = Figure("TargetBitrate")
- target_bitrate.addSubplot(['target_bitrate_bps'], "Time (s)", "Bitrate (bps)")
+ target_bitrate.AddSubplot(['target_bitrate_bps'], "Time (s)", "Bitrate (bps)")
# Select which figures to plot here.
figures = [receiver, detector_state, trendline_state, target_bitrate]
@@ -157,10 +157,10 @@ def main():
test_name = re.search(r'\.(\w+)', line).group(1)
if line.startswith("PLOT"):
try:
- (var_name, ssrc, alg_name, time, value) = parse_plot_line(line)
+ (var_name, ssrc, alg_name, time, value) = ParsePlotLine(line)
for f in figures:
# The sample will be ignored bv the figures that don't need it.
- f.addSample(var_name, ssrc, alg_name, time, value)
+ f.AddSample(var_name, ssrc, alg_name, time, value)
except ParsePlotLineException as e:
print e.reason
print e.line
@@ -168,8 +168,8 @@ def main():
# Plot figures.
for f in figures:
fig = plt.figure(f.name)
- f.plotFigure(fig)
- if save_figure:
+ f.PlotFigure(fig)
+ if SAVE_FIGURE:
fig.savefig(test_name + f.name + ".png")
plt.show()

Powered by Google App Engine
This is Rietveld 408576698