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

Can we add "y'all" to the normalizer conversion table? #635

Closed
MeanwhileMedia opened this issue Jan 7, 2022 · 4 comments
Closed

Can we add "y'all" to the normalizer conversion table? #635

MeanwhileMedia opened this issue Jan 7, 2022 · 4 comments

Comments

@MeanwhileMedia
Copy link

In my app i'm finding "y'all" to be a somewhat common contraction that isn't accounted for in normalizer.js:
https://github.com/NaturalNode/natural/blob/master/lib/natural/normalizers/normalizer.js#L34

@devinhedge
Copy link

Y'all better fix this or y'all fixin to git a whoopin bi the Southern Man.

#sarcasm #dialects

@titanism
Copy link

You should use something like expand-contractions before passing your String to the tokenizer.

See #533

@titanism
Copy link

Ref: stdlib-js/stdlib#496

@Hugo-ter-Doest
Copy link
Collaborator

Won't fix, is better solved with preprocessing.

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

4 participants