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

Add meta2.is as an extra prefix to outputs #35

Merged
merged 1 commit into from
Apr 9, 2024

Conversation

Ales-ibt
Copy link
Contributor

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

I have found the answer to why to use meta2.id in this module. I am using it now to name the outputs, otherwise, the name of input and output files is the same when more than one round of decontamination is needed, which makes the process crash.

Otherwise, the name of inout and output files are redundant in subsequent rounds of decontamination
Copy link
Contributor

@KateSakharova KateSakharova left a comment

Choose a reason for hiding this comment

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

Nice, thank you!

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 good to me!

@Ales-ibt Ales-ibt merged commit 3204918 into main Apr 9, 2024
6 checks passed
@Ales-ibt Ales-ibt deleted the fix/bwamem2decontnobams branch April 9, 2024 17:36
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