-
-
Notifications
You must be signed in to change notification settings - Fork 296
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
Fix word to be a prefix for term issue #798
base: main
Are you sure you want to change the base?
Conversation
@naira-petrosyan-m is attempting to deploy a commit to the OramaSearch Team on Vercel. A member of the Team first needs to authorize it. |
Hi @naira-petrosyan-m, thank you for opening this! May I ask you to merge |
@naira-petrosyan-m were you able to run tests locally? |
@micheleriva actually there are a lot of tests failing where I did not touch anything, I suppose maybe I have some missing parts. |
@micheleriva
I don't think the issue here is with Levenstein, maybe there is an issue somewhere else, that you need to take a look on. When you play with the test cases
this gives 0 results with my branch and 1 result with main branch, but should give 2 results logically ({ libelle: 'ABRICOT MOELLEUX' },
gives correct 2 results (my branch { libelle: 'ABRICOT MOELLEUX' }, I think there is an issue in _findLevenshtein function, but not sure, please check it when you can |
Issue described in #797