Skip to content

Include links to other projects? #36

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

Closed
thoppe opened this issue Dec 8, 2017 · 4 comments
Closed

Include links to other projects? #36

thoppe opened this issue Dec 8, 2017 · 4 comments

Comments

@thoppe
Copy link

thoppe commented Dec 8, 2017

flashtext is great, thank you for building, documenting, and writing a post about it!

I've incorporated it into our pipeline and saw a 60x speedup with respect to the regex matching we were doing. If you'd like to link back to our project (as proof yours is being used in the wild), feel free, or I can submit a PR for that. If not, thanks again for the quick drop-in library!

@vi3k6i5
Copy link
Owner

vi3k6i5 commented Dec 9, 2017 via email

@vi3k6i5
Copy link
Owner

vi3k6i5 commented Dec 9, 2017 via email

@thoppe
Copy link
Author

thoppe commented Dec 9, 2017

@vi3k6i5 thanks! We made it in house, like you, to solve some of the problems we had with our text. Our problem wasn't scale, but historical or corrupted data.

@vi3k6i5
Copy link
Owner

vi3k6i5 commented Jan 26, 2018

Closing as PR is there. Thanks 👍

@vi3k6i5 vi3k6i5 closed this as completed Jan 26, 2018
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

No branches or pull requests

2 participants