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

changes in FASTQC 0.12: Adjust default adapters and add total base count #64

Closed
wm75 opened this issue Jul 16, 2024 · 5 comments
Closed

Comments

@wm75
Copy link

wm75 commented Jul 16, 2024

v0.12 of fastqc from last year changed the default adapters it searches and now no longer reports SOLID adapters, but polyA and polyG instead.

see: https://github.com/s-andrews/FastQC/releases/tag/v0.12.0

The new release also now reports the total base count in basic stats, which would be good to have reflected in falco I guess.

Other changes in the fastqc release look less important at first glance (though maybe the deduplicated sequences line still present in falco will confuse people even more now that it got removed from fastqc).

@andrewdavidsmith
Copy link
Collaborator

Thanks for this. I hope to get to it soon.

@bgruening
Copy link

@andrewdavidsmith could we also get a new release? We would really push for falco instead of fastqc. But this and a new release is blocking us atm.

Thanks a lot.

@andrewdavidsmith
Copy link
Collaborator

@wm75 @bgruening I've taken care of the adapter and polyA/polyG stuff. Let me know if your use of falco depends on the total base count and/or the duplicated sequence thing. If it does, I'll try to prioritize it. Otherwise I'll make a new release of falco right away with the adapter/contaminant stuff updated.

@wm75
Copy link
Author

wm75 commented Aug 10, 2024

Thanks @andrewdavidsmith!
I think the adapter report is the most important change and a release containing this (and the bam rv-reads fix) would be great.
I don't mind the extra line in the duplicates plot at all, and the base count is a nice-to-have if you have the time to include it but not crucial.

@andrewdavidsmith
Copy link
Collaborator

andrewdavidsmith commented Aug 11, 2024

Ok, I made a release (v1.2.3) and it takes care of the adapter and contaminant lists issue, along with any changes since the previous release. If either of you @wm75 or @bgruening want to open a separate issue and just paste in those other suggestions, please do so and I'll get to it when I have time. The more bite-sized the work seems at a glance, the more likely I am to squeeze it into my schedule. However, if v1.2.3 has introduced any problems (I only tested it on 2 systems plus GH runners), then I can address it right away.

I'll also keep an eye on conda and if they don't auto-update the version in a couple days, I'll do it manually.

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

No branches or pull requests

3 participants