-
Notifications
You must be signed in to change notification settings - Fork 611
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
Vega 2 Sort #612
Vega 2 Sort #612
Conversation
- modify scale.sort() to produce scale's sort object - modify sort property to be consistent with Vega 2 (TODO: consider if we should move `sort` to become scale's property -- to be consistent with Vega2)
Actually there are still some incorrect cases for A) Dot Plot
although the output
The output visualization does not order the scale correctly B) Faceted Small Multiples
The
but the output is not well sorted |
However, similar visualizations from A) Dot Plot
B) Small Multiple
Not sure what makes things works correctly for barley.json, but not for movies.json I have inspected barley.json and make sure that the original data is not pre-sorted by |
Actually some variables in movies.json works correctly – for example, But some like Not sure what makes these sorting incorrect but this sounds more like a bug in lower-level stacks. |
Thanks for merging. I just appended changes to |
(part of #459)
sort.js
and no longer usesort-<field_name>
tables as data sourcescale.domain()
to include DataRef’s.sort
objectsort
property to be consistent with Vega 2 (but still leave it as a top-level vega-lite property)sort.name
=>sort.field
sort.aggregate
=>sort.op