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

Allows plotting GridSpaces inside Layouts in bokeh #1104

Merged
merged 3 commits into from Feb 6, 2017

Conversation

Projects
None yet
2 participants
@philippjfr
Member

philippjfr commented Feb 5, 2017

This PR makes the bokeh GridPlot and LayoutPlot fully composable so you can now plot multiple grids together. Here's two grids and a regular element in a layout:

%%opts Image [height=260]
(hv.GridSpace({(i, j): hv.Curve(range(i+j)) for i in range(1, 3) for j in range(2,4)}) +\
 hv.GridSpace({(i, j): hv.Curve(range(i+j)) for i in range(1, 3) for j in range(2,4)}) +
 hv.Image(np.random.rand(10,10)))

screen shot 2017-02-05 at 11 53 08 pm

The Composing_Data tutorial now also works for both matplotlib and bokeh:

screen shot 2017-02-06 at 2 48 12 am

Related: #381

@philippjfr philippjfr added bokeh bug feature and removed bug labels Feb 5, 2017

@philippjfr philippjfr added this to the v1.7.0 milestone Feb 6, 2017

@philippjfr

This comment has been minimized.

Member

philippjfr commented Feb 6, 2017

Hmm, the drop in coverage is odd. Will have to look into it.

@philippjfr

This comment has been minimized.

Member

philippjfr commented Feb 6, 2017

Now fixed, ready to review/merge.

@jlstevens

This comment has been minimized.

Member

jlstevens commented Feb 6, 2017

Unfortunately there now seem to be merge conflicts. Happy to merge once that is resolved and the tests have passed.

@philippjfr

This comment has been minimized.

Member

philippjfr commented Feb 6, 2017

First time I resolved merge issues directly on GitHub, ready to merge when passing again.

@jlstevens

This comment has been minimized.

Member

jlstevens commented Feb 6, 2017

Looks good and the tests are passing. Merging.

@jlstevens jlstevens merged commit f7c4b9d into master Feb 6, 2017

3 of 4 checks passed

s3-reference-data-cache Test data not cached, see details to rebuild.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.02%) to 78.169%
Details

@philippjfr philippjfr deleted the bokeh_grid_layout branch Feb 10, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment