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

Match exceeds max_l_dist when max_substitutions=0 is set #42

Open
markussteindl opened this issue Jul 11, 2022 · 1 comment
Open

Match exceeds max_l_dist when max_substitutions=0 is set #42

markussteindl opened this issue Jul 11, 2022 · 1 comment
Labels

Comments

@markussteindl
Copy link

find_near_matches('xyz', 'wz', max_l_dist=2)
works correctly and yields Match(start=1, end=2, dist=2, matched='z').

find_near_matches('xyz', 'wz', max_substitutions=0, max_l_dist=2)
does not work correctly since it yields

[{Match} Match(start=0, end=1, dist=2, matched='w'),
 {Match} Match(start=1, end=1, dist=2, matched='')]

The problem is that the substring 'xyz' and the matched part of the string 'w' have distance 3 which is larger than max_l_dist.

@taleinat taleinat added the bug label Jul 14, 2022
@taleinat
Copy link
Owner

Thanks for the report @Stonatus, that looks like a bug!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants