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

Reimplement bin using bin_start, bin_end #693

Merged
merged 5 commits into from
Oct 23, 2015
Merged

Reimplement bin using bin_start, bin_end #693

merged 5 commits into from
Oct 23, 2015

Conversation

kanitw
Copy link
Member

@kanitw kanitw commented Oct 23, 2015

  • Reimplement binning to use linear scale instead of ordinal scale based on @jheer’s example in Updated Bin transform. vega#388 using - use the new bin transform’s bin_end
  • Make scale.zero automatically determined rather than always true
    • always returns false for bin and time
    • Otherwise, return true for X, Y and SIZE

TODO

  • use bin_mid instead

@kanitw kanitw mentioned this pull request Oct 23, 2015
2 tasks
@kanitw
Copy link
Member Author

kanitw commented Oct 23, 2015

Okay. Things looks good now for histograms (and when replacing bar with point and line).
The area one still looks odd, but I am not sure how to make it looks better so I defer optimizing using bin with area for #694 .

vega-lite_gallery

@kanitw kanitw changed the title [WIP] Reimplement bin using bin_start, bin_end Reimplement bin using bin_start, bin_end Oct 23, 2015
@domoritz domoritz mentioned this pull request Oct 23, 2015
@@ -252,6 +253,81 @@ var EXAMPLES = [
},
'data': {'url': 'data/cars.json'}
}
},{
Copy link
Member

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.

Copy link
Member Author

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.

Copy link
Member

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.

domoritz added a commit that referenced this pull request Oct 23, 2015
Reimplement bin using `bin_start`, `bin_end`
@domoritz domoritz merged commit 64aa265 into master Oct 23, 2015
@domoritz domoritz deleted the kw/bin branch October 23, 2015 20:57
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