hello- "ping" is the name of the variable here (but perhaps i should change it to avoid confusion). in the function, the request is passed inside the try/except loop, which is nested in the for loop. so it will try making the request the # of times defined as tries. n is the result of the request, in this case either JSON or struct_time. it did retry in my testing.

i did test your suggested fix and it errors out with 'Response' object is not callable and AttributeError: 'bound_method' object has no attribute 'tm_mday'.

The line where get_request(5, io.receive_time()) is being invoked are first invoking io.recieve_time() and sending the resulting time structure as the second argument of get_request(). The second argument, ping then contains the result of io.recieve_time() through out. So it won't ever trigger an exception because all that's happening is the time structure in ping is being assigned to the variablen (unless something went so wrong that assignment fails).

You're seeing the 'Response' object is not callable error because my bug report didn't include the fix for line 87, whih looks like this:
response = get_request(5, requests.get(weather_url))

With the proposed change to get_requests() that activates retries, line 87 needs to change like this:
response = get_request(5, lambda : requests.get(weather_url))

I didn't realize this line was present until I submitted my pull request, which is why I didn't mention it in the bug report.

The lambda expression lets you delay the invocation of requests.get(weather_url) so that it also runs in the retry loop. It's needed because we can't send the bare reference to requests.get like we did for io.recieve_time, because the former needs an argument passed to it. Using the lambda expression lets us embed the argument into a new piece of code that will then be run in a retry loop.

This kind of thing is hard to test because it would require causing io.recieve_time() to fail the first time it is invoked but not the second time.

This might help for anyone who needs a refresher for passing functions as parameters/ variables in python.