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

[ENH] Word Cloud: Show bag of words weights when bag of words features available #486

Merged
merged 1 commit into from
Dec 18, 2019

Conversation

PrimozGodec
Copy link
Collaborator

@PrimozGodec PrimozGodec commented Dec 17, 2019

Issue

Bag-of-words weights should be shown in the Word Cloud widget. When TF-IDF is used bag-of-words weights should better highlight the importance of the word. It was also discussed at the last meeting.

Description of changes

With this PR bag-of-words, weights are used in the Word Cloud widget when available.

Includes
  • Code changes
  • Tests
  • Documentation

@PrimozGodec PrimozGodec changed the title Word Cloud: Show bag of words weights when bag of words features available [ENH] Word Cloud: Show bag of words weights when bag of words features available Dec 17, 2019
@codecov-io
Copy link

codecov-io commented Dec 17, 2019

Codecov Report

Merging #486 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #486      +/-   ##
==========================================
+ Coverage   61.86%   61.93%   +0.06%     
==========================================
  Files          59       59              
  Lines        6181     6197      +16     
  Branches      804      808       +4     
==========================================
+ Hits         3824     3838      +14     
- Misses       2221     2222       +1     
- Partials      136      137       +1

@ajdapretnar
Copy link
Collaborator

Implements #401.

@@ -11,9 +11,10 @@ Generates a word cloud from corpus.
**Outputs**

- Corpus: Documents that match the selection.
- Word: Selected word that can be used as query in [Concordance](concordance.md).
- Selected Word: Selected word that can be used as query in [Concordance](concordance.md).
- Word Counts: Word together withe their weights.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Word together withe their weights. --> Words and their weights.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@ajdapretnar ajdapretnar merged commit e3b58be into biolab:master Dec 18, 2019
@PrimozGodec PrimozGodec deleted the wordclod-bow branch December 18, 2019 15:39
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