nextstrain / mpox

Nextstrain build for mpox virus
https://nextstrain.org/mpox
MIT License
45 stars 19 forks source link

Shorten author name list #290

Closed corneliusroemer closed 5 days ago

corneliusroemer commented 1 week ago

Right now the author name list is very long, causing deteriorated functionality:

Brave Browser 2024-11-16 15 06 08

We should shorten with e.g. augur curate's functionality for shortening author names.

joverlee521 commented 6 days ago

@genehack asked a similar question for yellow fever on Slack, so summarizing discussion here

We already abbreviate authors as part of the ingest curate rule and output as the abbr_authors column. However, the full authors list is being used in the final Auspice output because augur export v2 is hard-coded to check for the author or authors field (ref).

Outside of updating augur export, the best path forward would be to switch authors to full_authors and abbr_authors to authors. Example in pathogen-repo-guide

corneliusroemer commented 6 days ago

Switching sounds good to me.

We should probably unhardcode authors in export though eventually.

jameshadfield commented 6 days ago

We should probably unhardcode authors in export though eventually.

Yes. My memory was that we used "authors" (in metadata TSVs) to represent "Roemer et al" etc, and augur export was designed around that. We've since switched to encoding the entire author list as "authors" without updating augur export.

genehack commented 5 days ago

☝️ created issue in augur repo for removing the hard-coded export.