Retry

[zmoon]

• retry decorator with backoff until max time (60 s) reached, a simplified (and currently inflexible) version of what backoff does, applied to the readers and cases of explicit requests usage
• add 10-s timeouts for explicit requests usage
• add timeout to the pytest step in CI

Resolves #21

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (edd5466) 97.44% compared to head (2158e60) 98.69%.

❗ Current head 2158e60 differs from pull request most recent head c8ced8e. Consider uploading reports for the commit c8ced8e to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #25      +/-   ##
==========================================
+ Coverage   97.44%   98.69%   +1.25%     
==========================================
  Files           3        4       +1     
  Lines         430      461      +31     
==========================================
+ Hits          419      455      +36     
+ Misses         11        6       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[zmoon]

Maybe just the requests.get usages that were sometimes failing, will have to check the logs more closely. I did see one case of urllib error though, will have to check if that came from read_csv or not.

Update: yes there was one case from get-data daily where read_csv had connection error (don't see the warning message for retry though, which is kind of weird). But the others were from requests.get.