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

docs: add tree to file list #190

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

docs: add tree to file list #190

wants to merge 1 commit into from

Conversation

j23414
Copy link
Contributor

@j23414 j23414 commented Apr 16, 2024

Minor edit to the pathogen.json file so that Nextclade Web can find the "tree.json" file. The Nextclade CLI call still works as expected.

Since the final augur tree might not be called "tree.json" could also add instructions to the user to modify the file name.

@j23414 j23414 deployed to refs/pull/190/merge April 16, 2024 15:04 — with GitHub Actions Active
@j23414 j23414 marked this pull request as draft April 16, 2024 15:41
@j23414
Copy link
Contributor Author

j23414 commented Apr 16, 2024

Still under discussion for where to place this or overall path for the Nextclade dataset docs. From slack:

We were hoping that making most files optional will make it easier for authors - you could start small and then add files gradually, improving the dataset. But now I see it also makes it a bit trickier to understand.

possibility benefit
only ref enables only alignment, nucleotide mutation calling, sequencing QC
ref + genome annotation adds gene level mutations and detail (translation and aa mutational calling)
ref + genome annotation + tree adds clades classification and phylogenetic placement
ref + tree uncertain use, perhaps only calling clades and tree placement

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.

1 participant