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

[WIP] Boxplot #371

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[WIP] Boxplot #371

wants to merge 1 commit into from

Conversation

mattwchun
Copy link
Contributor

@mattwchun mattwchun commented Apr 23, 2017

  • Update CompassQL Mark Type and Aggregation Function

  • Update default aggregation to enumerate [undefined, #'box-plot'] instead of [undefined, 'mean']

  • Add New Constraints

    • only one axis should be continuous( if there is another axis, it should be discrete)
    • aggregate = boxplox iff mark = boxplot
  • Update Rankings

…vega-lite. However, don’t know how some of the vega-lite types changed particularly what FacetedUnitSpec and Formula changed to.
@mattwchun
Copy link
Contributor Author

test cases

  • write a query with mark = ? with one Q-field where aggregate = {enum: [undefined, boxplot])

  • write a query with mark = ? with one N-field and one Q-field where aggregate = {enum: [undefined, boxplot])

  • write a query with mark = ? with two Q-fields where aggregate = {enum: [undefined, boxplot]) -- this one shouldn't show any boxplot because it's dumb

  • write a query with mark = ? with two Q-fields where aggregate = {enum: [undefined, boxplot]), bin = {enum: [false, true]) -- the case where one of them is binned would make sense

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.

1 participant