search

sterrenb / flutterhole

A third party Android application for the Pi-Hole® dashboard.
https://play.google.com/store/apps/details?id=sterrenburg.github.flutterhole
MIT License
400 stars 27 forks source link

Graph displays incorrectly #105

Open ansonchappell opened 4 years ago

ansonchappell commented 4 years ago

Describe the bug The graph looks weird Screenshot_20200805-064053

FelixTheAwesomeCat commented 4 years ago

Yep, same on my device! Can confirm Screenshot_20200806-114954

FelixTheAwesomeCat commented 4 years ago

@ansonchappell I noticed that the charts on the admin front page of pihole are also not working, could u check that on your browser? It might be a pihole bug?

FelixTheAwesomeCat commented 4 years ago

Flushing the LOGS fixed it for me (under settings in admin panel)

LordSimal commented 4 years ago

I got the same problem but for me the reason for this graph is the fact, that I have 20k+ requests.

Screenshot_20200817-230005

I would say a fix would be to show only 2k steps after 10k results or something like that and not show all the available steps in between

like what the web UI actually does Screenshot_20200817-230422

ansonchappell commented 4 years ago

Oh me too, maybe that's my issue

Anson Chappell

On Mon., Aug. 17, 2020, 15:03 Kevin Pfeifer, notifications@github.com wrote:

I got the same problem but for me the reason for this graph is the fact, that I have 20k+ requests.

[image: Screenshot_20200817-230005] https://user-images.githubusercontent.com/9105243/90444052-857a4b00-e0dd-11ea-877b-b818f1d3870f.jpg

I would say a fix would be to show only 2k steps after 10k results or something like that and not show all the available steps in between

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sterrenburg/flutterhole/issues/105#issuecomment-675114126, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQQKJOG463MDRV5FRFQT6O3SBGLLHANCNFSM4PVN4IQA .

sterrenb commented 4 years ago

I would say a fix would be to show only 2k steps after 10k results or something like that and not show all the available steps in between

That sounds like the solution! I believe the graph API has some limitations here, but will take a look.

ansonchappell commented 4 years ago

I just looked at the app and I'm displaying correctly now. Anson Chappell 867-446-3198 *Flickr http://www.flickr.com/photos/ansonchappell/ *YouTube http://www.youtube.com/user/ansonchappell Website https://www.ansonchappell.com/

On Wed, 19 Aug 2020 at 12:13, Thomas notifications@github.com wrote:

I would say a fix would be to show only 2k steps after 10k results or something like that and not show all the available steps in between

That sounds like the solution! I believe the graph API has some limitations here, but will take a look.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sterrenburg/flutterhole/issues/105#issuecomment-676581302, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQQKJOFXMACC72TY45VAJILSBQI23ANCNFSM4PVN4IQA .