search

eliasdabbas / advertools

advertools - online marketing productivity and analysis tools
https://advertools.readthedocs.io
MIT License
1.15k stars 213 forks source link

Update _yt_helpers.py #346

Closed DanielP77 closed 7 months ago

DanielP77 commented 7 months ago

dtype int translates to c datatype long, which is a 32-bit Integer on Windows. So, for very large channels, the total number of views exceeds the supported value range (try searching for channels with videos about "Cute cat", which will return channels with > 5bn views), leading to an OverflowError ("Python int too large to convert to C long"). Using dtype 'int64' explicitly uses a 64-bit Integer also on Windows machines and solves the issue.

eliasdabbas commented 7 months ago

@DanielP77 Great. Nice catch, and thanks for the fix