search

covidatlas / coronadatascraper

COVID-19 Coronavirus data scraped from government and curated data sources.
https://coronadatascraper.com
BSD 2-Clause "Simplified" License
364 stars 179 forks source link

add scraper for Orleans Parish closes #1000 #1032

Closed sglyon closed 4 years ago

sglyon commented 4 years ago

Summary

Adds scraper for Orleans Parish in US:LA

See issue #1000

This includes icu_current and hospitalized_current, which relies on #1030. It does not however include recovered or active. Not sure how we want to reconcile this as the current source for Orleans parish (via state scraper) does have those two fields. I think we'd need some sort of field merging.

This is my first contribution here, so if there are conventions I didn't follow please let me know!

jzohrab commented 4 years ago

Thanks @sglyon . I posted a msg to you in Slack: "I see that the source data has a bunch of fields, eg. {"OBJECTID":24,"Date":1585224000000,"NO_Cases":997,"NO_Deaths":46,"NO_State_Tests":null .... Should this scraper pull that information in as well?

Combining with the other sources is unfortunately tricky due to the architecture.

sglyon commented 4 years ago

yep, that makes sense. If the data is there we should grab it!

jzohrab commented 4 years ago

Hi @sglyon - apologies for letting this PR sit unattended. I was busy doing reporting and other work in Li, and lost sight of it for a moment. That sucks for contributors like you, so apologies again!

I don't wish to merge this into CDS, as we intend to shut this project down in favor of Li. I'm going to try to port this over to Li, I'll let you know how it goes. Unfortunately my time now is limited but I'll see what I can do. Cheers and regards, jz

jzohrab commented 4 years ago

Hi @sglyon - I've ported this over to Li in https://github.com/covidatlas/li/pull/352. Check out the PR comments and the code. I think it looks good. Thanks very much for the contribution and apologies for the delay, it's been a busy time. Cheers! jz