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

Canada Recovery Data #47

Closed
RKattoula opened this issue Mar 29, 2020 · 6 comments
Closed

Canada Recovery Data #47

RKattoula opened this issue Mar 29, 2020 · 6 comments

Comments

@RKattoula
Copy link

Not seeing recovery data for Canada, but it is being updated in the John Hopkins data.

Those are the only NA's I'm seeing. Great work on this - thanks a ton.

@anuveyatsu
Copy link
Member

anuveyatsu commented Mar 29, 2020

@RKattoula thanks for reporting this. It's happening due to inconsistency in the data files from the upstream:

Any suggestion would be useful 😄

@sympatheia-one
Copy link

Suggestions: There seem to be at least 3 options: (1) leave out to reflect the source and see if they will eventually update (2) "allocate" the total # of Recovered to the various provinces (for example in relation to their confirmed cases) or (3) just push the Revovered cases for Canada into a new set of rows where the Province/Region is labelled as something non-existing like RECOVERED AGGREGATE and which then will just have the Recovered numbers.

(1) is obviously the easiest...
(2) is inventing numbers that don't really exist and may not be useful for some
(3) [my preference] at least on a sum-per-country level we are back to the full picture

@anuveyatsu
Copy link
Member

@sgg70 option 3 sounds good to me

anuveyatsu added a commit that referenced this issue Mar 29, 2020
@anuveyatsu
Copy link
Member

Fixed. Please, check and close this issue.

@RKattoula @sgg70

@sympatheia-one
Copy link

hmm... just one question:
image
looking at the raw file, there is now a "Recovery" block with no real value, and then the "Recovery Aggregated", as suggested. I guess that first part can be omitted!?

@anuveyatsu
Copy link
Member

@sgg70 yes, the first one is the buggy data from the upstream that I've missed. Should be fixed now.

Gwardii pushed a commit to Gwardii/covid-19 that referenced this issue Mar 17, 2024
Gwardii pushed a commit to Gwardii/covid-19 that referenced this issue Mar 17, 2024
Gwardii pushed a commit to Gwardii/covid-19 that referenced this issue Mar 17, 2024
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

3 participants