vega / compassql

CompassQL Query Language for visualization recommendation.
Other
273 stars 43 forks source link

Change `omitTableWithOcclusion` => `omitTableWithOcclusionIfAddAutoCount` #179

Closed kanitw closed 8 years ago

kanitw commented 8 years ago

Without autoCount: true, it seems like this is not always the best idea.

kanitw commented 8 years ago

Let's change to omitTableWithOcclusionIfAutoCount and return false only if config.autoCount = true.

kanitw commented 8 years ago
kanitw commented 8 years ago
kanitw commented 8 years ago
{
  "spec": {
    "data": {"url": "data/cars.json"},
    "mark": "?",
    "encodings": [
      {
        "channel": "y",
        "field": "Cylinders",
        "type": "ordinal"
      },
      {
        "field": "Name",
        "type": "nominal",
        "channel": "?"
      }
    ],
    "config": {
      "overlay": {"line": true},
      "scale": {"useRawDomain": true},
      "facet": {"cell": {"width": 150,"height": 150}}
    }
  },
  "chooseBy": "effectiveness",
  "config": {
    "verbose": true
  }
}