search

cyclestreets / cyclestreets-r

An R interface to cyclestreets.net APIs
https://rpackage.cyclestreets.net/
GNU General Public License v3.0
27 stars 7 forks source link

timeout on GET() requests #91

Closed Robinlovelace closed 2 months ago

Robinlovelace commented 2 months ago

As discussed with @mvl22 and mentioned here https://github.com/nptscot/npt/pull/529#issuecomment-2324501355 sometimes batch() times-out. Not sure why but hypothesis is that adding something like httr::timeout(600) here will solve it:

https://github.com/cyclestreets/cyclestreets-r/blob/0aca7ef9bc09f6d28319f12f3298dca45e8c1c7c/R/batch.R#L210

Heads-up @wangzhao0217

Robinlovelace commented 2 months ago

Interestingly we already have something similar:

https://github.com/cyclestreets/cyclestreets-r/blob/0aca7ef9bc09f6d28319f12f3298dca45e8c1c7c/R/batch.R#L309