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

Allow configurable msm for fuzzy search #3174

Closed
wants to merge 2 commits into from

Conversation

romainruaud
Copy link
Collaborator

It was previously hardcoded to "100%", leading to some strange cases if the default minimum_should_match was not 100% but configured to something else.

Eg :

  • default MSM of "2<80%"
  • search for "open front dress" on a Venia Catalog => 12 results.
  • search again for "open front driss" => 0 results.

because the search with a typo triggered a fuzzy search that has an hardcoded value of 100% for minimum_should_match.

@romainruaud romainruaud force-pushed the fix_msm-fuzzy branch 4 times, most recently from 7b65329 to ca473fd Compare February 8, 2024 11:13
@romainruaud romainruaud requested a review from rbayet February 8, 2024 11:16
@rbayet rbayet changed the base branch from 2.11.x to 2.10.x February 26, 2024 09:52
@rbayet rbayet changed the base branch from 2.10.x to 2.11.x February 26, 2024 09:53
@rbayet
Copy link
Collaborator

rbayet commented Feb 26, 2024

Replaced by #3196 on 2.10.x

@rbayet rbayet closed this Feb 26, 2024
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.

2 participants