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

IMP: restructure dada2 invocation from R script to rpy2 #172

Open
colinvwood opened this issue Dec 3, 2024 · 1 comment
Open

IMP: restructure dada2 invocation from R script to rpy2 #172

colinvwood opened this issue Dec 3, 2024 · 1 comment
Assignees

Comments

@colinvwood
Copy link
Contributor

Currently we use an R script (q2_dada2/assets/run_dada.R) to run the dada2 pipeline for all four denoise methods in this plugin. Thus separate logic for each of these actions exists in each of the python functions and is also intermingled in the R script. Instead we should use the rpy2 library to interact with dada2's API within each method, increasing modularity. This will also make the plugin more maintainable because our team is more familiar with python than with R.

@colinvwood colinvwood self-assigned this Dec 3, 2024
@github-project-automation github-project-automation bot moved this to Backlog in 2025.4 🌻 Dec 3, 2024
@colinvwood colinvwood moved this from Backlog to In Development in 2025.4 🌻 Dec 3, 2024
@github-project-automation github-project-automation bot moved this to Needs Triage in QIIME 2 - Triage 🚑 Dec 3, 2024
@colinvwood
Copy link
Contributor Author

This should probably wait until after #169 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog
Development

No branches or pull requests

1 participant