vnstat-dashboard icon indicating copy to clipboard operation
vnstat-dashboard copied to clipboard

Published 20 hours ago • verified publisher icon alexandermarston

Add support for 5 minute data from vnstat 2.x - compatible with vnstat 1.x

Open tomangert opened this issue 5 years ago • 11 comments

Tested with vnstat 2.3 and vnstat 1.14 on Ubuntu 16.04 / Apache 2.4.39

tomangert avatar Jul 22 '19 22:07 tomangert

Add zoom and pan capability to 5min/hour/day graphs More fixes / cleanup

tomangert avatar Aug 08 '19 00:08 tomangert

@tomangert thanks for doing some work on this and getting it up to date. I'll have a look at this PR in detail tonight hopefully.

Does PR #36 need to be closed now this one is open?

alexandermarston avatar Aug 08 '19 07:08 alexandermarston

Yes 36 can be closed - changes from 36 are included in 37.

tomangert avatar Aug 08 '19 09:08 tomangert

A few comments with this one initially, @tomangert :

  • The content now appears to take out the whole page width whereas before content was centered, could we please revert this to how it was?
  • I'm not sure the use of being able to pan and zoom the graph, given the small dataset it is.
  • Could you revert the formatting of the time in the hourly list to its original (i.e.: '8pm')

alexandermarston avatar Aug 08 '19 16:08 alexandermarston

  1. OK.
  2. V2 allows you to configure the dataset size for each sampling interval. I have mine bumped up and it makes the pan/zoom pretty useful.
  3. I can use am/pm instead of 24hr time - but if you have more than 24 hours in the data you need the day. I can check the time range and suppress the day if not needed.

tomangert avatar Aug 08 '19 17:08 tomangert

@tomangert can you update your copy against the new changes and we can have a look what it looks like.

alexandermarston avatar Aug 08 '19 18:08 alexandermarston

ok - probably tonight. (gmt-4 here)

tomangert avatar Aug 08 '19 18:08 tomangert

Hey - any update on this?

Schokobecher avatar Oct 05 '19 15:10 Schokobecher

The history on this is a bit of a mess. The end result is 5 files with changes. Let me know if I should redo this in a new pull request.

tomangert avatar Oct 10 '19 20:10 tomangert

@tomangert this looks good. Have you got an example of this running with the 5 minute dataset?

alexandermarston avatar Oct 11 '19 09:10 alexandermarston

Example sent in an email.

The current merge request is the minimum changes to get something working.

Problems / enhancements left to deal with (~decreasing severity):

  1. vnstat2 keeps a lot more data. Current code displays it all on the initial graph. but frequently it is too much to see the individual bars. It needs to set the initial zoom level to show about the same as vnstat1 - or set number of time periods in config file - or compute on the fly based on screen size.
  2. Scroll-to-zoom and drag-to-pan help, but parameters need tweaking to allow more zoom range and finer control.
  3. Need to add help text for scroll/drag

The rest are not directly 5 minute/vnstat2 related and may just be my preferences.

  1. Logarithmic graph. This can be a config file option. My network is lightly loaded 90% of the time (~1% of bandwidth) Other times it is moderately loaded (interactive or streaming something) and occasionally there is something saturating the bandwidth (backup or big download or ...) . With a linear graph I can only see saturated vs next to nothing most of the time. A logarithmic graph shows a lot more nuance.
  2. Screen formatting. On medium to large screens the graphs are only using the middle half of the screen - about 1/4 of the screen is blank on both sides. For text that is useful to make it easier to read, but for graphs wider is better.

tomangert avatar Oct 11 '19 14:10 tomangert