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

Addition of lexical resource derived from wikipedia string-matching. #92

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

pseudomonas
Copy link

Addition of lexical resource derived from wikipedia string-matching.

Script to generate the lexical resource is included; has dependency on wikipedia-api package, though this is not required merely to use the prebuild resource (included)

Added matching routine that takes advantage of the lexical resource. The lexical resource is not case-or-accent normalised in case it will become useful at a future time to take advantage of case-variants in scoring matches.

…arch methods find a valid country, they will return it without trying the slower later methods. Speeds things up massively
@ctheune
Copy link
Collaborator

ctheune commented Mar 6, 2022

Ok this looks interesting. Can you add a few test cases to show that your code is working?

@ctheune
Copy link
Collaborator

ctheune commented Mar 6, 2022

Also, please note that I added black styling and rebase.

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.

None yet

3 participants