Skip to content

dependency: remove pycurl package#30771

Merged
adeebshihadeh merged 28 commits intocommaai:masterfrom
royjr:replace-pycurl
Dec 18, 2023
Merged

dependency: remove pycurl package#30771
adeebshihadeh merged 28 commits intocommaai:masterfrom
royjr:replace-pycurl

Conversation

@royjr
Copy link
Contributor

@royjr royjr commented Dec 17, 2023

may fix #30763

comma@comma-33e0a0c6:/data/openpilot$ python tools/lib/tests/test_caching.py
....
----------------------------------------------------------------------
Ran 4 tests in 3.145s

OK

@royjr royjr marked this pull request as draft December 17, 2023 06:02
@royjr royjr marked this pull request as ready for review December 17, 2023 08:59
@royjr
Copy link
Contributor Author

royjr commented Dec 17, 2023

fixed debug

ValueError: Format specifier missing precision

by setting precision value

get https://commadataci.blob.core.windows.net/openpilotci/0375fdf7b1ce594d/2019-06-13--08-32-25/3/qlog.bz2 {'Connection': 'keep-alive', 'Range': 'bytes=239824-239923'} 0.284 slow
get https://commadataci.blob.core.windows.net/openpilotci/0375fdf7b1ce594d/2019-06-13--08-32-25/3/qlog.bz2 {'Connection': 'keep-alive'} 0.548 slow

@adeebshihadeh adeebshihadeh merged commit 70624ff into commaai:master Dec 18, 2023
@adeebshihadeh
Copy link
Contributor

Thanks, this was one of the worse dependencies for native macOS support.

I just bumped up this one to $500 if you wanna try it - #30706.

@royjr royjr deleted the replace-pycurl branch December 18, 2023 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[$100 bounty] Remove pycurl usage

2 participants