cmu-delphi / epidatr

Delphi Epidata API R Client
https://cmu-delphi.github.io/epidatr/
Other
1 stars 5 forks source link

Aggregate lists of dates and/or geos #291

Open dsweber2 opened 3 weeks ago

dsweber2 commented 3 weeks ago

If someone hands us seq(as.Date("2014/09/04"), by = "day", length.out = 5), ideally we should change that into epirange("2014-09-04",2014-09-09). This helps the backend handle the result correctly. Doing a similar thing if every possible geo is detected.

May be better handled updating the backend to automatically do this for queries instead.

dshemetov commented 3 weeks ago

Already handled to a large degree server side though wouldn't be too hard to implement here too.

dsweber2 commented 2 weeks ago

Oh I guess that mostly just makes this better for visibility into the kinds of API requests we're getting then, rather than a concrete benefit to the user.