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

Active cases column #7

Open
ulklc opened this issue Mar 24, 2020 · 5 comments
Open

Active cases column #7

ulklc opened this issue Mar 24, 2020 · 5 comments

Comments

@ulklc
Copy link
Owner

ulklc commented Mar 24, 2020

Current structure already have confirmed, death and cured. I am thinking to add active column with this formula (confirmed - (death + cured) = active).
day | countryCode | countryName | region | lat | lon | confirmed | cured | death | *active*
What do you think ? Should we add that ?

@chrisjbillington
Copy link

I would say no, but I don't feel too strongly about it. It's redundant and easily calculated by the user, and its presence could be misleading - the user might think "If this is here, does that mean it is something different to confirmed - recovered - deaths?"

While we're at it, 'cured' is an unusual label for the recoveries column. I am guessing you are not a native English speaker, so to explain, 'cured' has connotations of the recovery being due to treatment, and is not really used for when patients recover spontaneously. So 'recovered' would be the more natural word.

@ulklc
Copy link
Owner Author

ulklc commented Mar 24, 2020

Yes, i am not a native eng speaker, and i agree. "recovered" label instead of "cured" making more sense. But it is requiring some code changes in crawler. Actually I can change it before more user start to use this repo.

@ulklc
Copy link
Owner Author

ulklc commented Mar 24, 2020

#8

@ivanMSC
Copy link

ivanMSC commented Mar 24, 2020

No to active column. Including it in the repo will only add unnecessary info, and my spreadsheet can only take so many bytes 😂
In fact, if you ask me, I would get rid of columns region, lat and lon, and put them on a separate table for reference.

@iandoug
Copy link

iandoug commented Mar 29, 2020

I'm native English, I also use "cured" simply because it's shorter than "recovered".
There is a point of view that it is the doctor's job to entertain the patient while nature does the healing, so the difference may be somewhat artificial.... especially in Covid-19 case where treatment is frequently aimed at the symptoms rather than the actual disease.

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

No branches or pull requests

4 participants