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

New fuzzy search algorithm for better results and perf #815

Merged
merged 2 commits into from Dec 29, 2015
Merged

Conversation

@kwm4385
Copy link
Contributor

kwm4385 commented Dec 18, 2015

Switched to a different fuzzy lib that'll give us better results and not nearly as laggy while typing.

@kwm4385
Copy link
Contributor Author

kwm4385 commented Dec 18, 2015

Memoizing the filter further reduced the lag from 320ms to 140ms in applicable situations. :)

@kwm4385 kwm4385 added the hs_staging label Dec 18, 2015
@tpetr tpetr added hs_qa labels Dec 28, 2015
@tpetr tpetr added this to the 0.4.8 milestone Dec 29, 2015
tpetr added a commit that referenced this pull request Dec 29, 2015
New fuzzy search algorithm for better results and perf
@tpetr tpetr merged commit 2a77487 into master Dec 29, 2015
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/push The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@tpetr tpetr removed hs_qa labels Dec 29, 2015
@ssalinas ssalinas deleted the new_fuzzy_search branch Apr 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.