-
-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show failed speedtests as 0? #43
Comments
@brycedjohnson Let me try. |
@brycedjohnson : Feature is now added in the |
Sure thing! Thanks!
…On Sat, Jul 25, 2020, 10:13 AM itsmesid ***@***.***> wrote:
@brycedjohnson <https://github.com/brycedjohnson> : Feature is now added
in the dev branch , wil you be able to test before it goes live ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#43 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AATWZ6IVSGWFUBRHJZA4KPDR5MADJANCNFSM4OF3UI7Q>
.
|
Now go to settings set an invalid speedtest id like Test by running |
@arevindh Tested and confirmed to work. Thanks for making it happen. |
I'm getting a error when I run the script |
@brycedjohnson can you update and try https://github.com/arevindh/pihole-speedtest/wiki/Updating--Speedtest-Mod |
@brycedjohnson You probably currently have speedtest-cli by sivel aka Python version in the web GUI. Meanwhile, speedtest-official.sh uses 'Speedtest CLI' from Ookla https://www.speedtest.net/apps/cli. |
@arevindh I did the update and still getting that syntax error. @karimnouh That sh script runs the official speedtest directly. I can run speedtest.sh script which runs the python script and that one works to get the speedtest, although I didn't get the no internet in the database when trying it with a different invalid server, or disconnected my pi from the internet during the test. Version from the web interface: Pi-hole v5.1.1 Web Interface v5.1 FTL v5.1 edit: After updating I forgot to checkout dev let me try that... |
@karimnouh Oops yes you were right. I had the Speedtest official installed at one point and it was working, but must have reverted back to the python script at some point. Also glossed over the first step somehow just now when reinstalling. Doh. @arevindh Tested it and it works now! Sorry about the confusion! |
Thanks for testing. |
@brycedjohnson @karimnouh: Line char or bar chart is better? ( Changed from bar to line chart along with this change). |
Line chart for sure! I saw that and liked it better.
…On Tue, Jul 28, 2020, 8:15 PM itsmesid ***@***.***> wrote:
@brycedjohnson <https://github.com/brycedjohnson> @karimnouh
<https://github.com/karimnouh>: Line char or bar chart is better? (
Changed from bar to line chart along with this change).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#43 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AATWZ6LVYFIDBR6XZBJ6LMTR56A5HANCNFSM4OF3UI7Q>
.
|
Agreed the line chart works way better for a time series. |
Might be nice to show failed speedtests as dl/up 0 in the graph? Had an internet outage today from for 11 hrs and looked back to see when the last successful test was run to see when it started and ended and if it was was working at all in between.
Thanks for the great work! Very useful!
The text was updated successfully, but these errors were encountered: