ncov icon indicating copy to clipboard operation
ncov copied to clipboard

Highlight currently circulating viruses

Open genehack opened this issue 1 year ago • 2 comments

Extracted from priorities meeting notes; issue created to track progress.

Ticket assignment based on lead in notes.

Related PRs

  • https://github.com/nextstrain/ncov/pull/1029
  • https://github.com/nextstrain/ncov/pull/1101
  • https://github.com/nextstrain/ncov/pull/934
  • https://github.com/nextstrain/ncov/pull/1129
  • https://github.com/nextstrain/ncov/pull/1132
  • https://github.com/nextstrain/auspice/issues/1946

genehack avatar Aug 16 '24 19:08 genehack

I've linked to related PRs in the issue description and updated individual PRs to clarify what's been tried, what's been done, and what is still on the table.

@trvrb do you think it's still worth pushing #1101 through?

victorlin avatar Oct 04 '24 18:10 victorlin

My sense is that including the additional context back to Wuhan is still useful, ie

Screenshot 2024-10-07 at 3 27 59 PM

despite the fact this context is now taking up 90% of the horizontal space on the page (it's still only a click away to zoom in to relevant circulating viruses). I think however that we will need to move to a "flu-like" system like https://github.com/nextstrain/ncov/pull/1129 in a year or so. At that time we should add a 2y option and also remove the 21L builds.

This leaves open/21L in a weird place. It should have gotten made alongside gisaid/21L, but now we think we'll be deprecating gisaid/21L in the coming months. I'd probably close https://github.com/nextstrain/ncov/pull/1129 in this case.

trvrb avatar Oct 07 '24 22:10 trvrb