Skip to content

Allow plot.kwic to plot multiple documents and keywords (fixes #78 and #65)#146

Closed
adamobeng wants to merge 9 commits into
quanteda:masterfrom
adamobeng:kwic-plot-documents
Closed

Allow plot.kwic to plot multiple documents and keywords (fixes #78 and #65)#146
adamobeng wants to merge 9 commits into
quanteda:masterfrom
adamobeng:kwic-plot-documents

Conversation

@adamobeng

Copy link
Copy Markdown
Contributor

No description provided.

adamobeng added 6 commits May 6, 2016 16:40
Requires ggplot and grid. If these are not present, will fall back to
the old kwic.plot function.

Signed-off-by: Adam Obeng <github@binaryeagle.com>
Removed legacy plotting function altogether.

Signed-off-by: Adam Obeng <github@binaryeagle.com>
Signed-off-by: Adam Obeng <github@binaryeagle.com>
Signed-off-by: Adam Obeng <github@binaryeagle.com>
Signed-off-by: Adam Obeng <github@binaryeagle.com>
Signed-off-by: Adam Obeng <github@binaryeagle.com>
@codecov-io

Copy link
Copy Markdown

Current coverage is 6.28%

Merging #146 into master will decrease coverage by -<.01%

@@            master      #146   diff @@
========================================
  Files           37        37          
  Lines         3373      3392    +19   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           213       213          
- Misses        3160      3179    +19   
  Partials         0         0          

Powered by Codecov. Last updated by e773a52...3fe6243

adamobeng added 3 commits May 11, 2016 10:12
Signed-off-by: Adam Obeng <github@binaryeagle.com>
Signed-off-by: Adam Obeng <github@binaryeagle.com>
Signed-off-by: Adam Obeng <github@binaryeagle.com>
@kbenoit

kbenoit commented May 24, 2016

Copy link
Copy Markdown
Collaborator

Merged in 6b6fe65, not sure why it did not auto-close this. Nice work.

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.

3 participants