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

Small typo under Adapter content section #202

Closed
SergeWielhouwer opened this issue Oct 21, 2024 · 1 comment · Fixed by #206
Closed

Small typo under Adapter content section #202

SergeWielhouwer opened this issue Oct 21, 2024 · 1 comment · Fixed by #206

Comments

@SergeWielhouwer
Copy link

Hi Ruben,

In the adapter content section of the HTML report, I noticed that there is a small typo at the start (v0.12.0).
For nanopore the the adapter mix (AMX) and ligation kit have overlapping adapter sequences and are therefore indistinguishable. Please consult the nanopore documentation for more information which adapters are used by your kit.

Really nice tool btw :). I am impressed by its performance, especially how ISA-L really helps when working with larger compressed FASTQ datasets.

Best,

Serge

@rhpvorderman
Copy link
Owner

Thanks! I will put this on my todo list.

Really nice tool btw :). I am impressed by its performance, especially how ISA-L really helps when working with larger compressed FASTQ datasets.

Thanks for the compliment. I noticed when I was doing some benchmarking in 2020 that a lot of tool runtime was spent on (de)compressing files. Simply setting the compression level for most tools to 1 saves so much time. Most tools actually seem like compression tools with a bioinformatic side effect at compression level 5.
ISA-L does wonders indeed. I had to do some work to get it running in an additional thread, but for someone as impatient as me it was certainly worth it in wall clock time!

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 a pull request may close this issue.

2 participants