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

Added check for subsequence exact matches #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kriskiil
Copy link

Additional exact matches are checked whether they are subsequences of previous matches. If that is the case, they are ignored.
Example: aroC 807 and 819 are subsequences of aroC 5.

Additional exact matches are checked whether they are subsequences of previous matches. If that is the case, they are ignored.
Example: aroC 807 and 819 are subsequences of aroC5.
@tseemann
Copy link
Owner

tseemann commented Aug 25, 2018

@kriskiil i think this can be resolved by just adding -culling_limit 1 to the blastn search command, like I do in abricate.

I thought I used to have this, but it isn't there. Some erroneous subsequence alleles were recently added (accidently) to pubMLST and I discovered this problem too.

I am going to add it back in 2.13

@tseemann tseemann self-assigned this Aug 25, 2018
@mjsull
Copy link

mjsull commented Jun 20, 2023

Hi Torsten, was this re-added, I seem to be having the same problem

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

Successfully merging this pull request may close these issues.

3 participants