-
Notifications
You must be signed in to change notification settings - Fork 1
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
Download URL used for speedtest does not support prerelease Fedora versions #1
Comments
Hi, I've tested myself using Fedora 37 with aarch64 architecture. And it seems working fine. |
Mm... Actually it's because I'm on Fedora 38 😅 Looks like the same file for prerelease versions can be found under |
Thanks for clarifying! |
I think there's still something wrong, but I'm not familiar with Python format strings, so... Here's the output I get:
Those speeds are impossibly low, and combined with the fact that they complete incredibly fast, I don't think it's downloading the |
Oh, also I've noticed But usually other tools (such as |
Here's my
|
diff --git a/speedtest.py b/speedtest.py
index 05e67bc..25e31ea 100644
--- a/speedtest.py
+++ b/speedtest.py
@@ -67,8 +67,8 @@ def speed_test(links, ostm, archi):
f.write(chunk)
done = int(30 * dl / int(total_length))
sys.stdout.write(
- f"\r{link} [%s%s] %s Kbps" % (
- '=' * done, ' ' * (30 - done), dl // (time.perf_counter() - start) / 1000))
+ f"\r{link} [%s%s] %s KiB/s" % (
+ '=' * done, ' ' * (30 - done), round(dl / (time.perf_counter() - start) / 1024, 2)))
speed = dl // (time.perf_counter() - start) / 100000
print('\n') With this change I get:
which looks clearer. (Of course this is unrelated to the problem of the speedtest not actually measuring anything useful...) |
Correct me, you've converted Kbps to KiB/s |
fedora-mirror-measure/speedtest.py
Line 40 in 04a44cc
The text was updated successfully, but these errors were encountered: