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

Issue 329480043: plot data automatically from ping

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years, 5 months ago by Liangzijie1437
Modified:
6 years, 5 months ago
Reviewers:
tonythomas
CC:
dkostic, kirill.sc, seregibenjamin, cynthiamao122_gmail.com, liangzijie1437_gmail.com, david.tomic11
Visibility:
Public.

Description

plot data automatically from ping draw plots together/separate draw latency together/separately draw_latency_plot script BUG=

Patch Set 1 #

Total comments: 1

Patch Set 2 : modify plot_results-from-ping #

Unified diffs Side-by-side diffs Delta from patch set Stats (+101 lines, -2 lines) Patch
M launch_lx3.py View 2 chunks +5 lines, -2 lines 0 comments Download
A utils/draw_latency_plot.py View 1 chunk +55 lines, -0 lines 0 comments Download
A utils/plot_results_from_ping.py View 1 1 chunk +41 lines, -0 lines 0 comments Download

Messages

Total messages: 9
Liangzijie1437
Please check the plot_results_from-ping file and some change in the launch-lx3 (much thanks to Tony). ...
6 years, 5 months ago (2017-10-16 20:49:22 UTC) #1
tonythomas
https://codereview.appspot.com/329480043/diff/1/utils/plot_results_from_ping.py File utils/plot_results_from_ping.py (right): https://codereview.appspot.com/329480043/diff/1/utils/plot_results_from_ping.py#newcode45 utils/plot_results_from_ping.py:45: print(validate_pings) No. Once you have this ping data as ...
6 years, 5 months ago (2017-10-16 20:51:21 UTC) #2
liangzijie1437_gmail.com
https://codereview.appspot.com/329480043 this one please. 2017-10-16 22:51 GMT+02:00 <01tonythomas@gmail.com>: > > https://codereview.appspot.com/329480043/diff/1/utils/plot_ > results_from_ping.py > File ...
6 years, 5 months ago (2017-10-16 22:21:33 UTC) #3
tonythomas
Did you test this one ? Can you paste the output as well ? Did ...
6 years, 5 months ago (2017-10-17 10:01:34 UTC) #4
liangzijie1437_gmail.com
Yes I test it, actually, now the problem is the X11 connection. It can not ...
6 years, 5 months ago (2017-10-17 10:34:16 UTC) #5
tonythomas
can you just saveplot to a file as I do in https://gits-15.sys.kth.se/REP/lx3/blob/master/utils/plot_ycsb_data_from_list.py#L48
6 years, 5 months ago (2017-10-17 10:36:58 UTC) #6
seregibenjamin
For the X11 issue, we have a workaround. Check lx3 readme "Issues". On 17 October ...
6 years, 5 months ago (2017-10-17 11:03:21 UTC) #7
liangzijie1437_gmail.com
I test it, now it can save the figure in the out folder after running ...
6 years, 5 months ago (2017-10-17 19:34:15 UTC) #8
tonythomas
6 years, 5 months ago (2017-10-21 12:36:10 UTC) #9
okey. merging it now. I resolved the conflicts.
Sign in to reply to this message.

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