-
Notifications
You must be signed in to change notification settings - Fork 80
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
display ANI in search results? #2001
Comments
ANI to
|
ahh, I'd kinda forgotten that. |
we could actually estimate ANI from
|
agreed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Once #1967 is merged, ANI will available in CSV files! 🎉
It is also available in
sourmash compare
matrix output, if--ani
is used. 🎉I don't think it is displayed anywhere else.
Do we want to add ANI to the
search
output, @bluegenes? I'm in favor - the search results are pretty sparse so I think we even have room for them.Not sure about gather, though. I think the k-mer overlap approaches makes more sense, maybe? But it would be nice to have as an option, maybe? 🤔
Maybe do it for search first, since I'm pretty sure that's a good idea, and then a separate PR (no hurry) for gather?
The text was updated successfully, but these errors were encountered: