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

MAINT: cluster-features-denovo tutorial needs new --p-strand param added #574

Closed
lizgehret opened this issue Feb 20, 2024 · 2 comments
Closed
Assignees
Labels
good first issue Good for newcomers

Comments

@lizgehret
Copy link
Member

The 2024.2 release included the following API change to cluster-features-denovo: qiime2/q2-vsearch#90

The vsearch tutorial should be updated to include the newly added --p-strand parameter.

@colinvwood this would be a good one to have @hagenjp work on while I'm OOO.

@lizgehret lizgehret added the good first issue Good for newcomers label Feb 20, 2024
@lizgehret lizgehret added this to 2024.5 Feb 20, 2024
@github-project-automation github-project-automation bot moved this to Backlog in 2024.5 Feb 20, 2024
@lizgehret lizgehret moved this from Backlog to In Analysis in 2024.5 Feb 20, 2024
@colinvwood colinvwood self-assigned this Feb 20, 2024
@hagenjp
Copy link
Contributor

hagenjp commented Mar 27, 2024

Hi @lizgehret, since this is an optional parameter that we are exposing and we do not have a use case in the tutorials for the non-default "both" option, is there any reason why this parameter should be added in the vsearch tutorial?

@lizgehret
Copy link
Member Author

Hey @hagenjp - in the tutorial, you can include the default --p-strand parameter in the cluster-features-denovo commands, then include a bit of text that describes the non-default use-case. This should be added because it keeps our tutorial up to date with the associated API change!

@hagenjp hagenjp removed this from 2024.5 May 29, 2024
@github-project-automation github-project-automation bot moved this to Backlog in 2024.10 🎃 May 29, 2024
@colinvwood colinvwood moved this from Backlog to In Development in 2024.10 🎃 Jun 13, 2024
@colinvwood colinvwood removed their assignment Jun 20, 2024
@hagenjp hagenjp moved this from In Development to Needs Review in 2024.10 🎃 Oct 1, 2024
@hagenjp hagenjp closed this as completed Oct 3, 2024
@hagenjp hagenjp moved this from Needs Review to Completed in 2024.10 🎃 Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
No open projects
Status: Completed
Development

No branches or pull requests

3 participants