EricJMarti / inventory-hunter

⚡️ Get notified as soon as your next CPU, GPU, or game console is in stock
MIT License
1.12k stars 264 forks source link

After most recent update, all scrapers throwing exception for refresh interval #180

Closed bgsmith17 closed 3 years ago

bgsmith17 commented 3 years ago

caught exception during request: 15.0 has type float, but expected one of: int, long

In the .yaml I have entered 15, no decimal place, but somewhere in the code it is getting cast to a float and now none of the scrapers work. Anyone got a fix? edit: on windows docker edit2: also on my raspberry pi 4

EricJMarti commented 3 years ago

@bgsmith17 I am unable to reproduce this locally so I'm curious what your yaml file looks like.

Just to be safe, I committed what I think is a fix: https://github.com/EricJMarti/inventory-hunter/commit/47495544927b376b298d181f97a901b2c7b8fbcd

bgsmith17 commented 3 years ago

Thanks for the quick response. Attached are the YAMLs I've been working with (converted to .txt format). amazon_rtx_3080.txt newegg_rtx_3080.txt ps5.txt

I'll update and let you know how it goes when I get home.

bgsmith17 commented 3 years ago

Still no dice after the update. The .yaml files haven't changed since before the update the other day, but I'll give one of the included ones a whirl and see if maybe that's the issue.

bgsmith17 commented 3 years ago

It appears that it was my .yaml files after all, after reinstalling and updating the packaged ones rather than the ones I had made from scratch the scrapers appear to be working. I'll let it run over night before closing the issue, but I think we're good.