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.

Sign up for GitHub

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

Jump to bottom

Replace inner product word matching with NLTK's concordance #8

Open
zhafen opened this issue Aug 28, 2020 · 1 comment
Open

Replace inner product word matching with NLTK's concordance #8

zhafen opened this issue Aug 28, 2020 · 1 comment
Labels
enhancement New feature or request

Comments

@zhafen
Copy link
Owner

zhafen commented Aug 28, 2020

Concordance is likely faster and more well-thought-out. Will need to check.
@zhafen zhafen added the enhancement New feature or request label Aug 28, 2020
@zhafen zhafen changed the title Replace inner word matching with NLTK's concordance Replace inner product word matching with NLTK's concordance Aug 28, 2020
@zhafen
Copy link
Owner Author

zhafen commented Aug 28, 2020 •
edited
Loading

Functions in cc to possibly modify:

  • publication.Publication.concept_projection; use nltk's Text.count instead
  • utils.uniquify_words, utils.match_words

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@zhafen