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

Fix issues from #41, #35, #44 #48

Merged
merged 5 commits into from
May 26, 2016
Merged

Fix issues from #41, #35, #44 #48

merged 5 commits into from
May 26, 2016

Conversation

j9recurses
Copy link
Contributor

Removes columns buttons from the UI that are ID fields.

@j9recurses j9recurses changed the title Fixes issue #41: Filter out ID columns from charts Fix issues from #41, #35, #44 May 25, 2016
@j9recurses
Copy link
Contributor Author

Closes issues #35, #44 relating to charting issues.

For #35- Column Display issues:
we are just taking the top 8 groups when displaying by rows. We could do something fancier where we look at count for each value in column then display however many fall within 50% of the max value (theres a lot different ways to approach this.

For #35- Displaying Group By
To reduce the dimensionality:
First, we remove any column group that is empty.
dataset = [ [x,x,x], [x,0,0,0], [x,0,0,1], [x,0,0,0] ]
in the above, items at index 1 and 3 are removed because they are empty

Then, we look a given row for a header. if the row has a sum of zero, we remove the row.
If not zero, then look at threshold --> 1% below the maxValue in the dataset, remove the row.

@jasonlally jasonlally merged commit 1d0ee03 into master May 26, 2016
@jasonlally
Copy link
Collaborator

👍

@jasonlally jasonlally deleted the limit_long_tail branch August 31, 2016 20:43
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.

2 participants