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

* Adding Option to Ignore Stop Words (English) from Tag Cloud #174

Conversation

ankit-singhaniya
Copy link
Contributor

Hi,
@chrismattmann

With this request, I am proposing a Feature Addition into existing Tag Cloud Panel of Banana.

Option to Ignore Stop Words

Description: The Tag Cloud panel is typically to be used with a field with only Keywords, this assumes that the Solr Field does not contain English Stop Words. Even if they did, the count of stop words will surely be prevalent than other keywords to be visualized. Problem with this is when this assumption is invalid on some field of solr index this panel is used on, it results in a Tag Cloud of Stop Words
Solution: Use of a List of Stop Words in English to reject words to be included in the cloud.
Approach: Adding an Option in Panel Settings to be Enabled to Ignore Stop Words from the Solr Field. This way, both the configurations of the panel could be used.

original tag cloud
Fig. 1: Original Tag Cloud Panel (and Output on some data)

modified tag cloud
Fig. 2: Modified Tag Cloud Panel (rejecting Stop Words)

original tag cloud settings panel
Fig. 3: Original Settings Page for Tag Cloud Panel

modified tag cloud settings panel
Fig. 4: Modified Settings Page for Tag Cloud Panel (adding a check-box for Stop Words)

@chrismattmann
Copy link

awesome work @ankit-singhaniya ! Guys if you need any help getting this in let me know!

@chrismattmann
Copy link

ping @dougmcclure can someone take a look at this?

@chrismattmann
Copy link

ping @dougmcclure can you take a look?

@chrismattmann
Copy link

ping can someone take a look at this? @athana

@athana
Copy link
Contributor

athana commented Dec 1, 2015

@chrismattmann @ankit-singhaniya Ok, I merged it to develop branch. Thanks for your contribution!

@athana athana closed this Dec 1, 2015
@chrismattmann
Copy link

thank you @athana !

@ankit-singhaniya
Copy link
Contributor Author

thank you @chrismattmann for bringing this up, and thanks @athana for taking look at the contribution!

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