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

typo fix : 'malyalam' to 'malayalam' #21

Closed
wants to merge 1 commit into from

Conversation

adamshamsudeen
Copy link

No description provided.

@@ -55,7 +55,7 @@ def __init__(self, lang: str):
LanguageTokenizer.__init__(self, lang)


class MalyalamTokenizer(LanguageTokenizer):
class MalayalamTokenizer(LanguageTokenizer):
Copy link
Owner

@goru001 goru001 Jul 3, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adamshamsudeen did you try and run inltk after making these changes? This won't work whenever malayalam language is being used because the malayalam model being loaded will look for class MalyalamTokenizer as it has reference stored by that name. You'll also have to also make changes to the class name here and export the model, store it on dropbox and update the link to model here

@goru001
Copy link
Owner

goru001 commented Jul 16, 2019

@adamshamsudeen I have created an issue for this here and have linked this PR there. Closing this PR for now. Thanks!

@goru001 goru001 closed this Jul 16, 2019
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

2 participants