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

Error in a complex bokeh plot #381

Closed
jlstevens opened this issue Dec 24, 2015 · 4 comments
Closed

Error in a complex bokeh plot #381

jlstevens opened this issue Dec 24, 2015 · 4 comments
Assignees
Labels
tag: backend: bokeh type: bug Something isn't correct or isn't working

Comments

@jlstevens
Copy link
Contributor

In a fairly complex plot I'm using for my work, I get issues when trying to display it with bokeh (works fine with matplotlib):

File "/Users/jstevens/Desktop/development/holoviews/holoviews/plotting/plot.py", line 188, in __getitem__
    if not self.drawn: self.handles['fig'] = self.initialize_plot()
  File "/Users/jstevens/Desktop/development/holoviews/holoviews/plotting/bokeh/plot.py", line 429, in initialize_plot
    subplots = subplot.initialize_plot(ranges=ranges, plots=shared_plots)
  File "/Users/jstevens/Desktop/development/holoviews/holoviews/plotting/bokeh/plot.py", line 558, in initialize_plot
    adjoined_plots.append(subplot.initialize_plot(ranges=ranges, plots=passed_plots))
TypeError: initialize_plot() got an unexpected keyword argument 'plots'

I realize this isn't very useful to reproduce the issue but maybe the traceback will be enough to figure out what is wrong. Otherwise, I'll take my data structure apart to find a simpler example that breaks.

@jlstevens jlstevens added type: bug Something isn't correct or isn't working tag: backend: bokeh labels Dec 24, 2015
@philippjfr
Copy link
Member

If you could at least provide the repr of the object you're displaying this should be much easier to debug.

My guess it's a GridSpace nested in a Layout which is not supported in bokeh. Should be a better exception message of course.

@jlstevens
Copy link
Contributor Author

Yes, it will be that:

:Layout
   .HoloMap.I                :HoloMap   [Time]
      :Overlay
         .Image.I :Image   [x,y]   (z)
         .Path.I  :Path   [x,y]
   .HoloMap.II               :HoloMap   [Time]
      :Curve   [x]   (z)
   .GridSpace.I              :GridSpace   [x,y]
      :Curve   [Time]   (z)
   .SpatioTemporalResponse.I :SpatioTemporalResponse   [x,y]   (z)

Shame bokeh doesn't currently support this and yes, an explicit error message saying so would be a good thing to have (if possible).

@philippjfr
Copy link
Member

This is now supported, closing.

Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tag: backend: bokeh type: bug Something isn't correct or isn't working
Projects
None yet
Development

No branches or pull requests

2 participants