-
Notifications
You must be signed in to change notification settings - Fork 608
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
Reimplement bin using bin_start
, bin_end
#693
Conversation
Okay. Things looks good now for histograms (and when replacing |
bin_start
, bin_end
bin_start
, bin_end
@@ -252,6 +253,81 @@ var EXAMPLES = [ | |||
}, | |||
'data': {'url': 'data/cars.json'} | |||
} | |||
},{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll put these in separate files some time later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think refering for exemplar ones in vega-editor make sense. For ones that are more visual test (like histogram with point, line -- we should make sure that it is easy to create ones)
Having it here, reduce the need to name the file nicely (and burden of renaming if the original name sucks, etc.) That said, this examples.js can become too damn long so I'm not against splitting it. Just wanna express potential downside of that here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have similar concerns. We will see after next week.
On Fri, Oct 23, 2015 at 1:58 PM, Kanit Wongsuphasawat <
notifications@github.com> wrote:
In gallery/examples.js
#693 (comment):@@ -252,6 +253,81 @@ var EXAMPLES = [
},
'data': {'url': 'data/cars.json'}
}
- },{
I think refering for exemplar ones in vega-editor make sense. For ones
that are more visual test (like histogram with point, line -- we should
make sure that it is easy to create ones)Having it here, reduce the need to name the file nicely (and burden of
renaming if the original name sucks, etc.) That said, this examples.js can
become too damn long so I'm not against splitting it. Just wanna express
potential downside of that here.—
Reply to this email directly or view it on GitHub
https://github.com/vega/vega-lite/pull/693/files#r42913620.
Reimplement bin using `bin_start`, `bin_end`
bin_end
scale.zero
automatically determined rather than alwaystrue
false
for bin and timeX
,Y
andSIZE
TODO
bin_mid
instead