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

New module/bwamem2 decont nobams #34

Merged
merged 6 commits into from
Apr 9, 2024
Merged

Conversation

Ales-ibt
Copy link
Contributor

@Ales-ibt Ales-ibt commented Apr 9, 2024

New decontamination module that generates decont fastq files on the fly without bam outputs.

@Ales-ibt Ales-ibt requested review from KateSakharova and chrisAta and removed request for KateSakharova April 9, 2024 09:19
Copy link
Member

@chrisAta chrisAta left a comment

Choose a reason for hiding this comment

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

Looks great to me thanks! Left a comment to start a little conversation about something we might want to consider as a team, but I don't really have any changes I think should be made

@Ales-ibt
Copy link
Contributor Author

Ales-ibt commented Apr 9, 2024

Thank you for your review. Merging branches now 🚀

@Ales-ibt Ales-ibt merged commit fa97e0d into main Apr 9, 2024
6 checks passed
@Ales-ibt Ales-ibt deleted the new_module/bwamem2_decont_nobams branch April 9, 2024 09:51
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.

3 participants