You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Context
This issue represents a list of minor odditites, bugs and gotchas I discovered while working on augur clades in PR #728
Despite the help indicating that nucleotide and/or amino-acid mutations are required, the node-data JSONs, when combined, must contain both muts and aa_muts keys for each node because the augur clades codes assumes their existence. Only one should be required.
Every node in the tree must have a corresponding entry in a node-data JSON, even if it has no mutations (this is asserted in NodeReader). We should allow nodes without information to be missing from node-data JSONs.
A single branch can define multiple mutations at the same position without an error being thrown, but each mutation overrides the previous and the results are unexpected. We should detect cases such as these and exit with an error.
#-prefixed lines in the clades TSV are not interpreted as comments, they're actually read as potentially valid clade definitions! You can mostly get away with this, as they define non-sensical clades, and thus don't cause any errors. I suggest we add comment='#' to pd.read_csv here.
related to ☝️ we should print warnings for each clade (in the supplied TSV) which isn't found in the tree.
The behaviour of augur clades means that if there are multiple nodes containing clade-defining mutations (i.e. the clade is polyphyletic), then we only annotate clades on the biggest monophyly. We should warn when situations like this arrise, or allow this to be relaxed. I expect it'll become common to want to define "clades" via a small set of constellation nCoV mutations, and expect polyphyletic colourings in Auspice.
The text was updated successfully, but these errors were encountered:
Multiple mutations at the same position on a single branch are now a
fatal error. Previous behaviour was to overwrite such mutations when
parsing. Suggested by #735.
Context
This issue represents a list of minor odditites, bugs and gotchas I discovered while working on
augur clades
in PR #728The text was updated successfully, but these errors were encountered: