-
Notifications
You must be signed in to change notification settings - Fork 30
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
Switch to single HTTP library. #109
Comments
Those were likely just the go-to libraries of the author(s) at the time. Right now, |
Hi thanks for responding,
Reasoning :
|
Addressing your points:
I'm not sure how true this is, since we will still have to maintain separate code for synchronous and asynchronous parts of the wrapper regardless. It will probably help some get started, but this is heavily dependent on an individual's familiarity with
I am all for using more popular frameworks/libraries if it means more people will be comfortable using and contributing to the project, but frankly, I'm not sure how true this statement is either. According to GitHub's dependency tracker, Furthermore, even if
You make a good point. As a quick and dirty test, if I make virtual environments (Python3.11, Ubuntu) only installing (1)
We see: Similarly, a new virtual environment with just a fresh install of Lastly, we should compare the performance of In summary:
If we can get better performance and reduce the size of our dependencies, I can see that as a compelling reason to switch. |
There are some interesting speed comparisons between and it looks like |
Hi, Thanks for creating this library. Is there any reason that you are including both
aiohttp
andrequests
.httpx
The text was updated successfully, but these errors were encountered: