Fixig issues in bwe plot dynamics.

Fixig issues in BWE dynamics plot scripts.

BUG=None

Committed: https://crrev.com/61050f67efbf3e3df2c6aa353db0d027be3eac4f
Cr-Commit-Position: refs/heads/master@{#14459}

[Gaetano Carlucci, 2016-09-22 08:45:56]

gaetano.carlucci@gmail.com changed reviewers:
+ stefan@webrtc.org

[Gaetano Carlucci, 2016-09-22 08:45:56]

This should fix the minor issues introduces with this
https://codereview.chromium.org/2296253002/

[stefan-webrtc, 2016-09-22 10:47:13]

lgtm

[stefan-webrtc, 2016-09-22 19:04:22]

The CQ bit was checked by stefan@webrtc.org

[commit-bot: I haz the power, 2016-09-22 19:04:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-22 19:18:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-22 19:18:47]

Try jobs failed on following builders:
  presubmit on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/8643)

[Gaetano Carlucci, 2016-09-23 12:48:03]

now gamma and offset dynamics can be seen with plt_dynamics.sh along with
packets loss, delay and throughput. At the moment the logging or other metrics
is orthogonal to this script.

[stefan-webrtc, 2016-09-23 13:49:41]

https://codereview.chromium.org/2360053003/diff/20001/webrtc/modules/remote_b...
File webrtc/modules/remote_bitrate_estimator/test/plot_dynamics.py (right):

https://codereview.chromium.org/2360053003/diff/20001/webrtc/modules/remote_b...
webrtc/modules/remote_bitrate_estimator/test/plot_dynamics.py:53: return
Is this still needed?

[Gaetano Carlucci, 2016-09-23 14:05:36]

I will remove that check

https://codereview.chromium.org/2360053003/diff/20001/webrtc/modules/remote_b...
File webrtc/modules/remote_bitrate_estimator/test/plot_dynamics.py (right):

https://codereview.chromium.org/2360053003/diff/20001/webrtc/modules/remote_b...
webrtc/modules/remote_bitrate_estimator/test/plot_dynamics.py:53: return
On 2016/09/23 13:49:41, stefan-webrtc (holmer) wrote:
> Is this still needed?

no. thanks

[Gaetano Carlucci, 2016-09-26 11:43:14]

The CQ bit was checked by gaetano.carlucci@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-26 11:43:26]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-26 11:59:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-26 11:59:36]

Dry run: Try jobs failed on following builders:
  presubmit on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/8686)

[stefan-webrtc, 2016-09-30 10:55:21]

Description was changed from

==========
fixig issues in bwe plot dynamics
==========

to

==========
Fixig issues in BWE dynamics plot scripts.
==========

[stefan-webrtc, 2016-09-30 11:01:24]

The CQ bit was checked by stefan@webrtc.org

[stefan-webrtc, 2016-09-30 11:01:25]

The patchset sent to the CQ was uploaded after l-g-t-m from stefan@webrtc.org
Link to the patchset: https://codereview.chromium.org/2360053003/#ps40001
(title: "removing not useful check")

[commit-bot: I haz the power, 2016-09-30 11:01:28]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-30 11:27:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-30 11:27:38]

Try jobs failed on following builders:
  presubmit on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/8833)

[stefan-webrtc, 2016-09-30 11:29:42]

Description was changed from

==========
Fixig issues in BWE dynamics plot scripts.
==========

to

==========
Fixig issues in BWE dynamics plot scripts.

BUG=None
==========

[stefan-webrtc, 2016-09-30 11:29:49]

The CQ bit was checked by stefan@webrtc.org

[commit-bot: I haz the power, 2016-09-30 11:29:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-30 13:29:56]

Description was changed from

==========
Fixig issues in BWE dynamics plot scripts.

BUG=None
==========

to

==========
Fixig issues in BWE dynamics plot scripts.

BUG=None
==========

[commit-bot: I haz the power, 2016-09-30 13:29:57]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-30 13:30:00]

Description was changed from

==========
Fixig issues in BWE dynamics plot scripts.

BUG=None
==========

to

==========
Fixig issues in BWE dynamics plot scripts.

BUG=None

Committed: https://crrev.com/61050f67efbf3e3df2c6aa353db0d027be3eac4f
Cr-Commit-Position: refs/heads/master@{#14459}
==========

[commit-bot: I haz the power, 2016-09-30 13:30:02]

Patchset 3 (id:??) landed as
https://crrev.com/61050f67efbf3e3df2c6aa353db0d027be3eac4f
Cr-Commit-Position: refs/heads/master@{#14459}