Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

alternative data source? #2

Open
pdobsan opened this issue Mar 24, 2020 · 2 comments
Open

alternative data source? #2

pdobsan opened this issue Mar 24, 2020 · 2 comments

Comments

@pdobsan
Copy link

pdobsan commented Mar 24, 2020

Thanks for your API/service. Among JSON API-s, I like it most so far.

Many people is disappointed by JSU's dropping the recovered category. As a consequence your source (pomper's) is probably going to drop it too. In the ongoing discussion I have come across an alternative data source which provides the recovered category and also looks like a clean CSV times series data-set: https://github.com/ulklc/covid19-timeseries

You might want to consider it.

Having said that, I am still hoping that JSU reconsiders since I need the state/province resolution.

@backtrackbaba
Copy link
Owner

Hi @pdobsan ,

Thank you for the suggestion. I'll check out the repo and its own sources. Maybe we could roll out v2 of the API with the new data source.

I agree that JHU, shouldn't have dropped the recovered category. I'm currently tackling an issue where newer countries are being added as null which is hindering my code. Fixing it right now.

Wanted to know, are you using our endpoints? Is there any feedback or suggestions that you would llike to pass on?

@pdobsan
Copy link
Author

pdobsan commented Mar 24, 2020

Wanted to know, are you using our endpoints?

Not yet, just have discovered it.

Is there any feedback or suggestions that you would llike to pass on?

Actually there is one, I have just discovered. When you provide historical data, I would much prefer an array of {date: , confiremed: ...} objects to the current format of object keyed with an ever increasing list of dates. Like it is in the original json source provided by @pomber. I don't see any reason to change that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants