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

APIs for both sources have issues #95

Open
prochazkaml opened this issue Mar 28, 2020 · 4 comments
Open

APIs for both sources have issues #95

prochazkaml opened this issue Mar 28, 2020 · 4 comments

Comments

@prochazkaml
Copy link

Hello,

I've tried to use the APIs for both sources, but each have their issues:

Source 1:

  • the statistics for China only display the statistics for the province Hubei, so it's stuck on 67.8k for more than a week (only an API issue, in the table, it's correct)

Source 2:

  • the reported countries are in the wrong order (China with 81.4k is above USA with 104k and Italy with 86.5k) - both an API and a table issue
  • the "top=X" parameter is ignored - it reports all of the ~200 countries (only an API issue)

The URL I use for fetching data is "https://corona-stats.online?format=json&top=10&source=X", where X is 1 or 2.

Since the first API doesn't report recovered cased, I tried to use the second API. But in the second API, it's not possible to get the data from the previous day. The first API had it and it was quite useful...

@hwiards
Copy link

hwiards commented Mar 28, 2020

Now China is on the last place in the order, when I query the data with:

curl 'https://corona-stats.online?minimal=true&source=2'

@onzyone
Copy link

onzyone commented Mar 29, 2020

Looks like the issue with the source=2 data is related to upstream data coming from here https://corona.lmao.ninja/countries

@hwiards
Copy link

hwiards commented Mar 29, 2020

Then the reqest URL should be changed to: https://corona.lmao.ninja/countries?sort=cases

EDIT: This part of the Issue seem to be fixed.

@sagarkarira
Copy link
Owner

@michalprochazka02

1.top param is not supported with json param.
2. recovered cases were removed by JHU. So the source API removed it too.

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

4 participants