Skip to content

Feat/upgrade nextclade#963

Merged
ivan-aksamentov merged 7 commits intomasterfrom
feat/upgrade-nextclade
Jun 28, 2022
Merged

Feat/upgrade nextclade#963
ivan-aksamentov merged 7 commits intomasterfrom
feat/upgrade-nextclade

Conversation

@rneher
Copy link
Copy Markdown
Member

@rneher rneher commented Jun 17, 2022

this updates the nextalign and nextclade calls to work with nextclade v2.

will require nextclade v2 in the image.

@rneher
Copy link
Copy Markdown
Member Author

rneher commented Jun 17, 2022

I started a trial build using the v2 docker image. results should show up here: https://nextstrain.org/staging/ncov/open/trial/nextclade-v2/REGION_NAME

@rneher
Copy link
Copy Markdown
Member Author

rneher commented Jun 18, 2022

The trial build completed without problem!

--output-dir {params.outdir} \
--output-basename {params.basename} \
--output-translations {params.output_translations} \
--output-fasta {params.uncompressed_alignment} \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to the changelog, version 2 of the Nextalign CLI can write compressed output files, perhaps rendering the uncompressed_alignment parameter unnecessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

3 participants