search

act-now-coalition / can-scrapers

MIT License
9 stars 13 forks source link

Iowa #41

Closed sglyon closed 3 years ago

christopherturner commented 3 years ago

I'll get to work on this one @sglyon!

christopherturner commented 3 years ago

Hey @sglyon – in the cmdc-tools Iowa scraper it looks like the data on hospitalizations is never passed along because there wasn't a way to support regions at the time.

For the ported scraper, should I port everything but this hospitalization data (function/comments from lines 47-49), or include it as well and make sure we don't call it from anywhere else until we deal with regions?

(In the second case, I'd likely mimic the PA code where I had a PennsylvaniaCasesDeaths class and then a PennsylvaniaHospitals that inherited from the former class).

sglyon commented 3 years ago

Hey @christopherturner thanks. I say we go for all of it!

Part of this PR will likely be to help us get up an running representing hospital regions.