Tweaks to axis labels and plot titles needed about prmon HOT 9 CLOSED

I just pushed a new branch named master-improve-plotting-script where I tried to address this issue. I tried to beautify the axis/legend labels, properly assign the units etc. This also addresses the plotting issues mentioned in #64.

Some examples are:

./prmon_plot.py --xvar wtime --yvar vmem,pss,rss --stacked

./prmon_plot.py --xvar wtime --yvar utime,stime,wtime

./prmon_plot.py --xvar wtime --yvar utime,stime --diff

./prmon_plot.py --xvar wtime --yvar rx_bytes,tx_bytes --diff

Could you give it a shot and let me know? Let me tag @graeme-a-stewart in case he also wants to comment.

Many thanks.

-s

from prmon.

@amete - thanks! My suggestion is that one can somehow scale the y-axis into a fixed unit size, e.g., GB or kB/s. When I see memory in "kB * 1e8" I have to do quite some mental gymnastics to realise that 0.2 = 20GB.

BTW, I like a lot plotting the time in relative seconds instead of the actual GMT clock time - much more useful.

from prmon.

Thanks @graeme-a-stewart. I guess we can easily pass a multiplier (for both x and y axes) or allow the user to choose from a list of possible options (kb, mb, gb etc.). Prior gives more freedom but makes it a bit cumbersome to decide what units we should write to the label while the latter is more restrictive but eases the second issue. I believe the mainline usage would be just to convert either memory/data-transfer or time from the default units to more meaningful ones (i.e. kb2gb and sec2min). So, perhaps the second option would suffice?

from prmon.

I just implemented and push the updated script that allows the user to change the units of the x/y-axes. The usage is like:

./prmon_plot.py --xunit MIN --yunit GB:

./prmon_plot.py --xunit MIN --yunit GB --diff:

I can add a further protection that checks the consistency between the variable and the unit, i.e. throw an error if one tries --xvar pss --xunit SEC etc. but I'm not super sure if this is really necessary, maybe I'm being over cautious?

-s

from prmon.

I am in favour of letting the user to choose the unit (any default is fine if the user can override it).
I was wondering if we can just get rid of " obtained from PrMon output", it makes the title unnecessarily long.

from prmon.

Sure, I can strip that part to make the title short. Do we want to mention PrMon in some part of the plot?

from prmon.

I don't see the need, personally.

from prmon.

Cool, I just pushed the changes w/ the shortened title.

from prmon.

Fixed with #66

from prmon.