AdGuardHome icon indicating copy to clipboard operation
AdGuardHome copied to clipboard

Dashboard Graphs became daily instead of Hourly in v0.107.44

Open target123 opened this issue 1 year ago • 7 comments

Prerequisites

Platform (OS and CPU architecture)

Linux, ARMv7

Installation

GitHub releases or script from README

Setup

On a router, DHCP is handled by AdGuard Home

AdGuard Home version

v0.107.44

Action

Replace the following command with the one you're calling or a description of the failing action:

nslookup -debug -type=a 'www.example.com' '$YOUR_AGH_ADDRESS'

Expected result

Graphs used to contain hourly data where you can see device activity graphed every hour for the last several days.

Allowing you to see detailed patterns of usage - (e.g. sleeping / wake times, when someone is home or not)

Actual result

Graph details in the dashboard contain less details and information compared to the previous versions.

It used to be the case that each data point in the graph represents an hour. Now, each data point represents 1 day - making the chart less detailed.

Hope that this change can be reverted as detailed graphs provide great insights and value to several use cases.

Additional information and/or screenshots

image

target123 avatar Feb 07 '24 01:02 target123

@target123, in v0.107.44

Statistics for 7 days displayed as 168 hours on the dashboard.

As a workaround, you can set the statistics retention interval to 167 hours on the General settings page.

schzhn avatar Feb 07 '24 10:02 schzhn

@target123, in v0.107.44

Statistics for 7 days displayed as 168 hours on the dashboard.

As a workaround, you can set the statistics retention interval to 167 hours on the General settings page.

Why not 168 Hours? Since 24*7=168? @schzhn

target123 avatar Feb 07 '24 16:02 target123

Fixed
 -  Statistics for 7 days displayed as 168 hours on the dashboard.

If the interval is 168 hours or more, statistics are displayed by day.

Related #6595

schzhn avatar Feb 07 '24 17:02 schzhn

Confirming that the bug is present on multiple platforms, tested on several Windows, FreeBSD, and ARM, Systems.

target123 avatar Feb 08 '24 04:02 target123

On the contrary, the bug has been fixed.

schzhn avatar Feb 08 '24 11:02 schzhn

This hasn't been fixed, the graphs are clearly displayed in the daily format, no detailed information as it's highly summarised for the seven day period. This is a change in behaviour from all previous builds and a new issue

Image below is taken from an ARM instance

image

joadamou avatar Feb 08 '24 17:02 joadamou

@schzhn, the issue here is the incorrect solution of the original issue. The backend should send hourly data for interval ≤ 168 hours, while the UI should not use the backend units when showing the number of days/hours in the statistics cards. See the behavior in the older releases.

ainar-g avatar Feb 08 '24 18:02 ainar-g

Bug seems to remain in v0.107.45

target123 avatar Mar 07 '24 03:03 target123

Fixed in edge and upcoming beta release

schzhn avatar Mar 13 '24 13:03 schzhn