Draw plots on percentiles, use pandas to create boxplots

Draw plots on percentiles, use pandas to create boxplots


BUG=REP-

[tonythomas, 2017-10-10 18:48:06]

This is not perfect yet, but shows separate box plots for 50th, 99th and max
latency rather than in one plot. 

I think its quite tough to show it in one plot.

[seregibenjamin, 2017-10-10 22:11:08]

Good to see a progress on this.

On 10 October 2017 at 20:48, <01tonythomas@gmail.com> wrote:

> Reviewers: ,
>
> Message:
> This is not perfect yet, but shows separate box plots for 50th, 99th and
> max latency rather than in one plot.
>
> I think its quite tough to show it in one plot.
>
> Description:
> Draw plots on percentiles, use pandas to create boxplots
>
>
> BUG=REP-
>
> Please review this at https://codereview.appspot.com/333840043/
>
> Affected files (+127, -19 lines):
>   M api.py
>   M experiments/setups/1x3.2cores.vanilla.json
>   M launch_lx3.py
>   A utils/plot_ycsb_data_from_list.py
>   A utils/read_yscb_data_from_folder.py
>
>
>

[david.tomic11, 2017-10-14 15:05:25]

Review comments and give check so i can merge this

https://codereview.appspot.com/333840043/diff/100001/api.py
File api.py (right):

https://codereview.appspot.com/333840043/diff/100001/api.py#newcode146
api.py:146: return get_yscb_data_from_folder(output_folder)
not sure why you use so many function calls? 

Would be more readable if it would just me implemented directly in the parent
function. But doesnt affect functionality though.

https://codereview.appspot.com/333840043/diff/100001/launch_lx3.py
File launch_lx3.py (right):

https://codereview.appspot.com/333840043/diff/100001/launch_lx3.py#newcode152
launch_lx3.py:152: 'label': 'thread count',
you sure the label shouldn't be jitter_level?

[tonythomas, 2017-10-14 15:18:18]

https://codereview.appspot.com/333840043/diff/100001/launch_lx3.py
File launch_lx3.py (right):

https://codereview.appspot.com/333840043/diff/100001/launch_lx3.py#newcode152
launch_lx3.py:152: 'label': 'thread count',
On 2017/10/14 15:05:25, david.tomic11 wrote:
> you sure the label shouldn't be jitter_level?

:D of course

[tonythomas, 2017-10-14 15:29:52]

merged. danke

--
Tony Thomas
https://mediawiki.org/wiki/User:01tonythomas
--

On Sat, Oct 14, 2017 at 5:18 PM, <01tonythomas@gmail.com> wrote:

>
> https://codereview.appspot.com/333840043/diff/100001/launch_lx3.py
> File launch_lx3.py (right):
>
> https://codereview.appspot.com/333840043/diff/100001/launch_
> lx3.py#newcode152
> launch_lx3.py:152: 'label': 'thread count',
> On 2017/10/14 15:05:25, david.tomic11 wrote:
>
>> you sure the label shouldn't be jitter_level?
>>
>
> :D of course
>
> https://codereview.appspot.com/333840043/
>

[david.tomic11, 2017-10-14 15:31:14]

bitte

On 14 October 2017 at 17:29, Tony Thomas <01tonythomas@gmail.com> wrote:

> merged. danke
>
> --
> Tony Thomas
> https://mediawiki.org/wiki/User:01tonythomas
> --
>
> On Sat, Oct 14, 2017 at 5:18 PM, <01tonythomas@gmail.com> wrote:
>
>>
>> https://codereview.appspot.com/333840043/diff/100001/launch_lx3.py
>> File launch_lx3.py (right):
>>
>> https://codereview.appspot.com/333840043/diff/100001/launch_
>> lx3.py#newcode152
>> launch_lx3.py:152: 'label': 'thread count',
>> On 2017/10/14 15:05:25, david.tomic11 wrote:
>>
>>> you sure the label shouldn't be jitter_level?
>>>
>>
>> :D of course
>>
>> https://codereview.appspot.com/333840043/
>>
>
>