-
Notifications
You must be signed in to change notification settings - Fork 145
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
[FEATURE REQUEST] Genome redundancy estimations compatible with introns #2337
Comments
Hey Tom, I think this is a great idea. Thank you for proposing it. It would be immensely helpful if you used default anvi'o files to characterize a genome with the default way of doing it, and with the proposed way of doing it and show the difference. So we can benchmark our code against those numbers while we use the same genome for testing. That would be a great help! |
Perfect. We are currently testing this on a series of MAGs, and as soon as we identify a good example where the difference between the two approaches is significant, we will share here that one as a FASTA and our estimated redundancy results. |
A contigs-db would be much better! :) |
Here is the data for testing (one genome, as FASTA and as a contigs-db, as well as the directory for our HMM collection (19 markers), and protein sequences for the hits). In that genome, completion is ~90%, and our redundancy estimate is ~90% with the classic mode and 0% with the correction, as all redundant hits are in the same contig! Tom |
Holy crap. This is really a very compelling case! :) Thank you, Tom. |
Well, it was the most compelling one, so it does not reflect all situations! If need be I can provide a another genome example with more mixed signals. Thank you in advance for looking into the code base with this feature in mind |
Yes, a mixed case would also help. It will likely forever stay as a flag since we can't get used to the idea that we treat these SCGs in a very special way by default. It would ruin everything phylogenomics :( |
Here is the second example. Tom |
The need
In the case of introns (eukaryotes and some of their viruses), the estimated genomic redundancy can be highly inflated, wrongly indicating that the quality is bad.
The solution
In those situations, the same HMM within a single copy core gene collection will tend to have multiple hits in the same contig. Our solution would be to consider contigs when estimating the redundancy of genomes, and only count 1 when the same HMM has multiple its in the same contig. Simple.
Programs that might provide a new flag are the following:
anvi-run-hmms
anvi-summarize
anvi-estimate-genome-completeness
I expect that a flag only for anvi-run-hmms might be the most simple solution, however anvi'o dev wizards know best and if they consider this request, I am looking forward to see how they elect to deal with it.
Or MAYBE, this strategy should become the norm instead of an alternative, as it would not at all impact quantifying true redundancy in intron-less genomes (redundant hits in multiple contigs) while improving the metric for intron-enriched genomes.
Beneficiaries
This would be beneficial to only a small number of people for now (those that dare to work on eukaryotes and their viruses with anvi'o), however better dealing with introns is certainly a step in the right direction given the ambition of anvi'o :)
At the Genoscope, we are willing to test the new ability and provide feedback to the anvi'o devs, and possibly beyond if results look promising
The text was updated successfully, but these errors were encountered: