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

Enhancement: Add bbmap suite for read preprocessing #39

Open
natgiot opened this issue Jun 17, 2022 · 1 comment
Open

Enhancement: Add bbmap suite for read preprocessing #39

natgiot opened this issue Jun 17, 2022 · 1 comment
Labels
enhancement New feature or request LW low priority priority for the LW developers

Comments

@natgiot
Copy link

natgiot commented Jun 17, 2022

BBmap is available here:
https://sourceforge.net/projects/bbmap/
it has been published in PloS One:
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC5657622/
and adopted by a wide community, including the JGI (here is a guide in their website for bbmerge: https://jgi.doe.gov/data-and-tools/software-tools/bbtools/bb-tools-user-guide/bbmerge-guide/)

Depending on how the suite of tools may be integrated in Pema, merging can be achieved through bbmerge, but additional steps (trimming, adapter removal) may also be handled with the same package in a very fast and efficient way.

Thank you for considering adding the tool, it appears to handle better than pandaseq the merging step of fully overlapping reads (insert size equal to read length cases)!

@hariszaf hariszaf added the enhancement New feature or request label Jun 18, 2022
@hariszaf hariszaf self-assigned this Jun 18, 2022
@hariszaf hariszaf removed their assignment Jun 13, 2023
@hariszaf
Copy link
Owner

This issue is to replace the merging function with a novel one that would invoke the BBmap tool.

As a first step, a bbmap function needs to be added in the preprocess.bds script.

Then an if statement should be added in pema_latest.bds and a parameter in the parameter files asking for the user which merging approach to use.

@kmexter kmexter added the LW low priority priority for the LW developers label Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request LW low priority priority for the LW developers
Projects
None yet
Development

No branches or pull requests

3 participants