Skip to content
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

NameError: name 'info_size' is not defined #12

Closed
itd opened this issue May 31, 2015 · 1 comment
Closed

NameError: name 'info_size' is not defined #12

itd opened this issue May 31, 2015 · 1 comment

Comments

@itd
Copy link

itd commented May 31, 2015

Just FYI... I cloned master (f7179f1), ran the script, and got this error:

# apt-select.py
Getting list of mirrors ... done.
Testing 73 mirror(s) [73/73] 100%
Traceback (most recent call last):
  File "/root/bin/apt-select.py", line 245, in <module>
    if info_size == 0:
NameError: name 'info_size' is not defined
@jblakeman
Copy link
Owner

I'm having trouble reproducing this error. It seems the only possible cause is if there were an empty dictionary of latency measurements. The culprit for no successful round trip measurements could be network latency so high that it exceeds the TCP socket timeout (2.5 seconds). This script is probably not much use if the machine's latency to all servers is over 2.5 seconds. In that case, an improvement of < 150 ms would likely be insignificant, and downloads would be a crawl regardless of the server they use.

A check for uncommon connectivity issues has been added in 9ee835d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants