Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(387)

Issue 339950043: plot RTT(with jitters) and scores of 1 node

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 months, 2 weeks ago by xinyue
Modified:
1 month, 3 weeks ago
Reviewers:
david.tomic11, tonythomas, liangzijie1437
CC:
dkostic, kirill.sc, seregibenjamin, cynthiamao122_gmail.com, liangzijie1437_gmail.com
Visibility:
Public.

Description

minior error and fix the timestamp precision in x axis BUG=

Patch Set 1 #

Patch Set 2 : take one node, cas1.3 for example, see the relation between score and jitter, generate them in 1 pl… #

Total comments: 8

Patch Set 3 : adjust timestamp of score, change to 3 digits after decimal point when generate log when link laten… #

Total comments: 1

Patch Set 4 : done the jitter with score plot #

Patch Set 5 : fix some errors of the code #

Patch Set 6 : remove redundancy comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -1 line) Patch
M utils/ds_score_plots/plot_jitter_host_to_host.py View 1 2 3 4 5 1 chunk +8 lines, -1 line 0 comments Download

Messages

Total messages: 9
xinyue
in order to explain the score we got better, take one node as an example ...
2 months, 2 weeks ago (2017-12-08 23:52:26 UTC) #1
tonythomas
Can you reply here with a sample plot you got ? -- Tony Thomas https://mediawiki.org/wiki/User:01tonythomas ...
2 months, 2 weeks ago (2017-12-09 10:14:05 UTC) #2
tonythomas
Alright. its taking good shape. I saw the result yesterday already, but adjust these things ...
2 months, 2 weeks ago (2017-12-09 10:21:43 UTC) #3
xinyue
https://codereview.appspot.com/339950043/diff/20001/launch_lx3.py File launch_lx3.py (right): https://codereview.appspot.com/339950043/diff/20001/launch_lx3.py#newcode495 launch_lx3.py:495: # plot_host_to_host_jitter(conf, save_dir=unique_exp_folder) On 2017/12/09 10:21:43, tonythomas wrote: > ...
2 months, 2 weeks ago (2017-12-09 19:01:31 UTC) #4
david.tomic11
On 2017/12/09 19:01:31, xinyue wrote: > https://codereview.appspot.com/339950043/diff/20001/launch_lx3.py > File launch_lx3.py (right): > > https://codereview.appspot.com/339950043/diff/20001/launch_lx3.py#newcode495 > ...
2 months, 2 weeks ago (2017-12-09 19:07:03 UTC) #5
Liangzijie1437
https://codereview.appspot.com/339950043/diff/40001/utils/ds_score_plots/plot_jitter_host_to_host.py File utils/ds_score_plots/plot_jitter_host_to_host.py (left): https://codereview.appspot.com/339950043/diff/40001/utils/ds_score_plots/plot_jitter_host_to_host.py#oldcode51 utils/ds_score_plots/plot_jitter_host_to_host.py:51: from h1.2 to s1, h1.2 and to h1.3 should ...
2 months, 2 weeks ago (2017-12-10 22:50:09 UTC) #6
xinyue
2 months, 1 week ago (2017-12-12 13:26:25 UTC) #7
xinyue
2 months, 1 week ago (2017-12-12 13:26:25 UTC) #8
david.tomic11
2 months, 1 week ago (2017-12-12 14:41:27 UTC) #9
On 2017/12/12 13:26:25, xinyue wrote:

You can create a loop for this instead of manually changing the nodes you read
from.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 204d58d