-
Notifications
You must be signed in to change notification settings - Fork 16
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
Average network speed include server latency #45
Comments
Yes. The average speed takes into account the response time(includes server latency), and the response size. Personally, i think the new api's provided by okhttp would be more accurate in terms of time taken and average speed. Also you can now track time on a granular level (dns lookup time, connection establishment time, and response time). The api's are not final yet, but this seems promising |
It is final now as of 3.11. Will be happy to send a PR if you are ok and we can agree on the approach. |
Sure, that would be great :) |
hi, @bangarharshit did you make the adjustment ? i looking for a similar solution |
No, I never did that because basic OkHttp events sufficed for me. this library needs your app to have large number of network calls so as to use moving window to find the network speed. This was not true for my app. I can make this fix but will be interested to know what is your use case? |
it's not a good idea to get download speed from the response body using EventLisetener until and unless you have a very big response body. we measured network speed from this and the value we are getting here is inaccurate. Let me know if anyone can measure downstream speed successfully. Our Average Network Body Length = 1.6k Byte Our Average Network Body Download Duration = 15 msThe Speed we are getting with this is very slow (0.00000594Mbps)
|
For slow network calls, the average speed includes endpoint latency. Now we have info about connection start in okhttp 3.9 - EventLisetener, do you guys think it will be useful?
The text was updated successfully, but these errors were encountered: