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

Streamline TF-Idf Api #73

Closed
thanasions opened this issue Oct 22, 2018 · 0 comments
Closed

Streamline TF-Idf Api #73

thanasions opened this issue Oct 22, 2018 · 0 comments
Assignees
@thanasions
Labels
enhancement New feature or request
Projects
pyGrams

Comments

@thanasions
Copy link
Contributor

At the moment there is a method that does term extraction from a single doc, with tests for it, which is wrong.

We need to public methods, one get all terms and one being get terms for a list of doc_ids
@thanasions thanasions added the enhancement New feature or request label Oct 22, 2018
@thanasions thanasions self-assigned this Oct 22, 2018
@thanasions thanasions added this to To do in pyGrams via automation Oct 22, 2018
@thanasions thanasions mentioned this issue Nov 15, 2018
Closed
@thanasions thanasions moved this from To do to In progress in pyGrams Nov 15, 2018
thanasions added a commit that referenced this issue Nov 15, 2018
@thanasions
small refactor to pass tests as usual. Closes #73
ed804df
thanasions added a commit that referenced this issue Nov 16, 2018
@thanasions
Merge pull request #84 from datasciencecampus/73_refactor_tfidf
18ecfac 
small refactor tfidf. Closes #73
pyGrams automation moved this from In progress to Done Nov 16, 2018
@thanasions thanasions mentioned this issue Nov 16, 2018
Closed
@thanasions thanasions mentioned this issue Mar 15, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thanasions thanasions

Labels
enhancement New feature or request
Projects
No open projects
pyGrams
  
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@thanasions