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

China region vs China country #30

Open
ivan-aksamentov opened this issue Sep 25, 2023 · 2 comments
Open

China region vs China country #30

ivan-aksamentov opened this issue Sep 25, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@ivan-aksamentov
Copy link
Member

In the web app, currently there are 2 "China"s in the list of locations. One is region ("China") and another is country ("CHN"). We need to disambiguate these.

Are these still different?

Options:

  • remove region
  • remove country
  • rename region to "China region" or similar
@ivan-aksamentov ivan-aksamentov added the bug Something isn't working label Sep 25, 2023
@huddlej
Copy link
Contributor

huddlej commented Sep 25, 2023

From a user perspective, I would want to be able to filter for China as both a region and a country. China is one of the few countries that have so many people that they get treated as their own "region". We recently updated SARS-CoV-2 subsampling logic to treat India like China in this sense.

An example of when I'd filter for China as region or country is in the seasonal flu H3N2 builds where I want to know what is circulating in China compared to Europe and I also want to compare China with other countries in Asia like South Korea. The main reason for keeping China as both region and country in the Auspice context is that Auspice filters AND across metadata columns instead of ORing.

I like that the frequencies app already allows mixed selection of "region" and "country" (ORing across metadata columns). In the current context of the app, I can tell that the three-letter codes represent countries and the other geographic options (e.g., "Europe") represent regions. I wish that the app was aware of the distinction between different geographic resolutions, though, and communicate that to users akin to how Auspice supports filters for different resolutions:

image

For example, I'd prefer that the frequencies app displayed "South Korea" as an entry among a list of all countries instead of the current "KOR" among a list of ungrouped geographic labels.

@huddlej
Copy link
Contributor

huddlej commented Sep 25, 2023

...which I see you're already doing in #27, @ivan-aksamentov 😄

I guess the comment above can be my 👍🏻 for the new features!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants