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

Different results based on genomes in reference list #57

Closed
aaronmussig opened this issue Dec 3, 2019 · 3 comments
Closed

Different results based on genomes in reference list #57

aaronmussig opened this issue Dec 3, 2019 · 3 comments

Comments

@aaronmussig
Copy link

Hello,

I was testing fastANI 1.2 and came across the same issue presented in #46 where list comparisons give different results to individual comparisons. I've included the genomes used to generate the below results here: fastani.zip.

The query list option gives a slightly different ANI and AF for U_77353_genomic and GCA_002499525 in the A vs. B direction compared to running it individually. This was tested against a fresh download of the 1.2 binary. This is the minimum subset of reference genomes which results in a different ANI/AF. Thanks for your work!

Running --ql vs --rl:

List A List B A vs B B vs A
U_77353_genomic GCA_002499525 99.2949 216/259 99.1235 221/288
GCF_002787055 79.0849 149/259 79.2057 144/565
GCF_000723185 76.748 52/259 76.4391 60/539

Running -q vs -r:

Genome A Genome B A vs B B vs A
U_77353_genomic GCA_002499525 99.1834 217/259 99.1235 221/288
U_77353_genomic GCF_002787055 79.0849 149/259 79.2057 144/565
U_77353_genomic GCF_000723185 76.748 52/259 76.4391 60/539
@cjain7
Copy link
Member

cjain7 commented Dec 5, 2019

I thought this was fixed in #46, but may be not. I'll check this and get back to you.

@cjain7
Copy link
Member

cjain7 commented Jan 19, 2020

Hi @aaronmussig

Sorting of one vector in my code was not deterministic, i've fixed that bug now. I was able to reproduce the above results, now they appear correct.
Thanks for sharing detailed output and files!

If you want to test at your end, please use the latest code from master branch.

@aaronmussig
Copy link
Author

Hi @cjain7 thanks for looking into this, it's much appreciated! It looks fine on my end now.

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

2 participants