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

Added downsample_columns operation #903

Merged
merged 4 commits into from Oct 5, 2016

Conversation

Projects
None yet
3 participants
@philippjfr
Copy link
Contributor

philippjfr commented Oct 5, 2016

No description provided.

Philipp Rudiger added some commits Oct 5, 2016

Philipp Rudiger Philipp Rudiger
Philipp Rudiger Philipp Rudiger
@jlstevens

This comment has been minimized.

Copy link
Contributor

jlstevens commented Oct 5, 2016

Looks good and I am much happier with the new name. Merging.

@jlstevens jlstevens merged commit 47b5582 into master Oct 5, 2016

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details

@philippjfr philippjfr deleted the downsample_operation branch Oct 14, 2016

@jbednar

This comment has been minimized.

Copy link
Contributor

jbednar commented on holoviews/operation/element.py in 5f20ea5 Apr 18, 2017

@philippjfr, where is plot supposed to be defined? This line is raising an error on current code using decimate, and I can't see how it could ever work.

This comment has been minimized.

Copy link
Contributor

philippjfr replied Apr 18, 2017

True, should be element.

This comment has been minimized.

Copy link
Contributor

jbednar replied Apr 18, 2017

Can you please test that? I already tried using element by adding decimate(points) to the end of census-hv.ipynb, but that didn't work for me either.

This comment has been minimized.

Copy link
Contributor

philippjfr replied Apr 18, 2017

Are you using a dask dataframe?

This comment has been minimized.

Copy link
Contributor

jbednar replied Apr 18, 2017

Yes.

This comment has been minimized.

Copy link
Contributor

jbednar replied Apr 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.