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

Consistent serotype-genotype column names and auspice color titles #51

Merged
merged 5 commits into from
May 21, 2024

Conversation

j23414
Copy link
Contributor

@j23414 j23414 commented May 20, 2024

Description of proposed changes

For Dengue serotype, genotype, and the various methods we employ to derive them for a specific strain (NCBI, Nextclade, augur clades (Nextstrain)), this PR is an implementation of consistent metadata column names and auspice color titles following the plan in #41 (comment)

Related issue(s)

Checklist

Link to test builds from this branch:

genome E gene
all all/genome all/E
denv1 denv1/genome denv1/E
denv2 denv2/genome denv2/E
denv3 denv3/genome denv3/E
denv4 denv4/genome denv4/E

or for a visual summary of the default color titles for the different trees:

@j23414 j23414 linked an issue May 20, 2024 that may be closed by this pull request
@j23414 j23414 changed the title Consistent serotype genotype column names Consistent serotype-genotype column names and auspice color titles May 20, 2024
@j23414 j23414 marked this pull request as ready for review May 21, 2024 17:00
@j23414 j23414 requested a review from a team May 21, 2024 17:00
Consistently names the serotype from Genbank fields as `serotype_genbank`
in the metadata and `Serotype (Genbank metadata)` in the auspice tree.
Consistently names the genotype Nextclade assignment as `genotype_nextclade`
in the metadata and `Dengue Genotype (Nextclade)` in the auspice tree.
During genome/dengue_all workflows, clade membership represents "Serotype (Nextstrain)""
While genome/dengue_denvX workflows, clade_membership represents the more detailed "Dengue Genotype (Nextstrain)"
For E gene builds that contain all serotypes, color by Serotype to match the
full genome builds that contain all serotypes.
@j23414 j23414 force-pushed the consistent-serotype-genotype-column-names branch from ce361ce to f941515 Compare May 21, 2024 21:26
@j23414 j23414 merged commit 57b57d1 into main May 21, 2024
32 checks passed
@j23414 j23414 deleted the consistent-serotype-genotype-column-names branch May 21, 2024 21:34
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

Successfully merging this pull request may close these issues.

Rename "subtype" to "genotype"
2 participants