-
Notifications
You must be signed in to change notification settings - Fork 241
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
Nice work! But the application is having some error.! #3
Comments
Fix resolved the issue, vote for closing it |
Open
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Great work indeed! But there are some issues with your approach. You are trying to store a large chunck of data which is not a good approach. It would be better to use the similarity criteria like Jacard's Similarity etc. Apart from that there are issues with the application's Stop button.
Nevertheless, Great effort and clean Code!!
The text was updated successfully, but these errors were encountered: