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

refactor vv code to use the new api (subset of docs) #83

Closed
thanasions opened this issue Nov 15, 2018 · 0 comments
Closed

refactor vv code to use the new api (subset of docs) #83

thanasions opened this issue Nov 15, 2018 · 0 comments
Labels
bug Something isn't working
Projects
pyGrams

Comments

@thanasions
Copy link
Contributor

refactor the vv so it trains the tf-idf matrix from >1000 docs and gets results by row only. That can be done after #73
@thanasions thanasions added this to To do in pyGrams via automation Nov 15, 2018
@thanasions thanasions added the bug Something isn't working label Nov 15, 2018
@thanasions thanasions moved this from To do to In progress in pyGrams Nov 16, 2018
@thanasions thanasions moved this from In progress to Needs review in pyGrams Nov 16, 2018
IanGrimstead added a commit that referenced this issue Nov 16, 2018
@IanGrimstead
Added Levenshtein module (#83)
4b1ceaf
IanGrimstead added a commit that referenced this issue Nov 16, 2018
@IanGrimstead
Allegedly corrected Levenshtein dependency (#83)
73e61c0
thanasions added a commit that referenced this issue Nov 16, 2018
@thanasions
Merge pull request #87 from datasciencecampus/83_vv_new_api
52e8e41 
vv tests with new API closes #83
pyGrams automation moved this from Needs review to Done Nov 16, 2018
@thanasions thanasions moved this from Done to Reviewer approved in pyGrams Nov 16, 2018
@thanasions thanasions moved this from Reviewer approved to Done in pyGrams Nov 16, 2018
thanasions added a commit that referenced this issue Nov 16, 2018
@thanasions
vv tests with new API closes #83
d3a5d9e
IanGrimstead pushed a commit that referenced this issue Nov 21, 2018
@thanasions @IanGrimstead
85_optimize_docset (#90)
fdaa737 
* vv tests with new API closes #83
* now it is O(n)! Resolves #86
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
No open projects
pyGrams
  
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@thanasions