-
Notifications
You must be signed in to change notification settings - Fork 413
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
Implement Kondrak n-gram similarity #15
Comments
How about using the Lucene implementation ? |
@MpoMp Sounds like a good idea. Based on my interpretation of the Apache license, I assume we would just need to make sure to include the Apache license header and indicate that we changed the code (to fit this library's constraints)? Despite being an ASF member and my familiarity with Lucene and its respective projects, I'm not an expert in licenses and copying the actual code. (To introduce myself, I'm a member of the team that ported this library to .NET, and a member of the Lucene.NET PMC.) cc @jamesmblair |
Disregard my deleted comment. What I meant was, isn't this the same as src/main/java/info/debatty/java/stringsimilarity/NGram.java? |
Looks like I misinterpreted the issue in the first place. As @tdebatty mentions in the README:
Which probably refers to the algorithms described on page 8 here. |
No description provided.
The text was updated successfully, but these errors were encountered: