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 feature request: Integrate NCBI FCS-adaptor into funannotate clean? #759

Open
IanDMedeiros opened this issue Aug 8, 2022 · 8 comments

Comments

@IanDMedeiros
Copy link
Contributor

Now that NCBI had released their internal adapter contamination screening tool for public use (https://ncbiinsights.ncbi.nlm.nih.gov/2022/07/28/fcs-beta-tool/), would it be possible to add an option for FCS-adaptor to be run as part of funannotate clean?

@hyphaltip
Copy link
Collaborator

patches always welcome - we also put more cleaning stuff in AAFTF stajichlab/AAFTF#7 - one issue I saw was at leats in the version NCBI had at beginning requires huge database setup and in-memoryruns so this may not be useable expect on HPCs so need to be clear.

@IanDMedeiros
Copy link
Contributor Author

I think the huge database is only for FCS-GX; the FCS-adaptor tool is small and runs quickly.

@hyphaltip
Copy link
Collaborator

oh that would be better - yeah that would be good to add in - just I think @nextgenusfs was thinking of separating assembly cleaning from annotation but let's see if we can get this integrated in. at least in AAFTF there is a vectrim step we can augment.

@nextgenusfs
Copy link
Owner

I think fine to add it in both places as long as install dependencies don't cause build issues.

@nextgenusfs
Copy link
Owner

I might recant about including: this looks to be a shell script running a container in either docker or singularity. https://github.com/ncbi/fcs/blob/main/dist/run_fcsadaptor.sh.

That won't package in conda very well I don't think.

On the surface it appears to do what we have been doing for awhile in AAFTF with a blast for adapters and automatically cleaning them. So would be easier to move that aaftf code here than deal with the docker/container issue.

@nextgenusfs
Copy link
Owner

For reference, we could integrate our "vecscreen" contig cleaner in funannotate clean, it seems to be doing very similar to what fcs-adapter is attempting to do (ie find contamination, remove and split contigs, etc). https://github.com/stajichlab/AAFTF/blob/main/AAFTF/vecscreen.py

@hyphaltip
Copy link
Collaborator

hyphaltip commented Aug 23, 2022 via email

@nextgenusfs
Copy link
Owner

ncbi/fcs#5

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