outbreak-info / biothings_covid19

Biothings plugin for JHU CSSE COVID-19 cases
6 stars 5 forks source link

Add unit tests, modularize covid19 parser for API #13

Open flaneuse opened 4 years ago

flaneuse commented 4 years ago
flaneuse commented 4 years ago

note from @gkarthik : "_numIncrease is negative for countries like Spain. JHU didn’t bother to correct this but I’m unsure on what to do here. For _rolling , I added a check to exclude any -ve values while computing the average using _numIncrease so that should fix that bit"

clarification from me: the _numIncrease threshold was arbitrarily set; values can be negative if there are corrections to daily totals the next day. ideally, this threshold would be inversely proportional to the administrative level, so counties should see much smaller corrections compared to countries. large negative numIncreases should throw a warning rather than an error-- just a cue to examine the data to make sure there wasn't something weird going on with the geographic assignment, for instance.

gkarthik commented 4 years ago

Potentially relevant issue outbreak-info/outbreak.info#132.