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

Add Nextclade to ingest #7

Merged
merged 4 commits into from
Oct 13, 2023
Merged

Add Nextclade to ingest #7

merged 4 commits into from
Oct 13, 2023

Conversation

joverlee521
Copy link
Contributor

@joverlee521 joverlee521 commented Oct 10, 2023

Description of proposed changes

Adds template for optional rules to run Nextclade as a part of the ingest workflow.
The Nextclade rules are optional because not every pathogen will have a Nextclade dataset to be able to run Nextclade as part of ingest.

This wraps up the "core" ingest workflow that produces the curated metadata and sequences.
Maybe not, we should probably add Biosample data to the ingest template.

Add rules for running Nextclade as a part of the ingest workflow. These
rules are optional because not every pathogen will have a Nextclade
dataset to be able to run Nextclade as a part of ingest.
joverlee521 and others added 3 commits October 11, 2023 16:33
The shell script for joining the metadata and Nextclade outputs is taken
from @j23414's work in nextstrain/mpox#207

Co-authored-by: Jennifer Chang <[email protected]>
Stop hard-coding the expected `seqName` ID field used from Nextclade
in case it get's renamed to something different in the user provided
Nextclade column map.
Adds the default rule to rename the subset metadata TSV to the final
metadata TSV if the Nextclade parameters are not defined in the config.

This allows users to run the core ingest workflow with the simplest
command of `nextstrain build .`
@joverlee521 joverlee521 force-pushed the add-nextclade-to-ingest branch from c7cc97c to 798c516 Compare October 11, 2023 23:37
@joverlee521
Copy link
Contributor Author

I kept this PR open to make sure it aligned with changes made in nextstrain/mpox#207.
Merging now since the monkeypox PR has been merged.

@joverlee521 joverlee521 merged commit d49aeeb into main Oct 13, 2023
@joverlee521 joverlee521 deleted the add-nextclade-to-ingest branch October 13, 2023 23:06
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