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

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 days, 23 hours ago by xinyue
Modified:
16 hours, 21 minutes 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
Unified diffs Side-by-side diffs Delta from patch set Stats (+37 lines, -31 lines) Patch
M utils/ds_score_plots/extract_data_from_log_file.py View 1 2 1 chunk +1 line, -2 lines 0 comments Download
M utils/ds_score_plots/plot_both_score_jitter.py View 1 2 5 chunks +23 lines, -20 lines 0 comments Download
M utils/ds_score_plots/plot_jitter_host_to_host.py View 1 2 1 chunk +12 lines, -8 lines 1 comment Download
M utils/link_latency_update.py View 1 2 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6
xinyue
in order to explain the score we got better, take one node as an example ...
2 days, 15 hours 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 days, 4 hours 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 days, 4 hours 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: > ...
1 day, 20 hours 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 > ...
1 day, 20 hours ago (2017-12-09 19:07:03 UTC) #5
Liangzijie1437
16 hours, 21 minutes ago (2017-12-10 22:50:09 UTC) #6
https://codereview.appspot.com/339950043/diff/40001/utils/ds_score_plots/plot...
File utils/ds_score_plots/plot_jitter_host_to_host.py (left):

https://codereview.appspot.com/339950043/diff/40001/utils/ds_score_plots/plot...
utils/ds_score_plots/plot_jitter_host_to_host.py:51: 
from h1.2 to s1, h1.2 and  to h1.3 should be in the same plot.
the same for from h1.1 and h1.3
Sign in to reply to this message.

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