Added comments to bokeh backend

holoviz · Feb 8, 2016 · 555f0d1 · 555f0d1
1 parent 175491d
commit 555f0d1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/holoviews/plotting/bokeh/__init__.py b/holoviews/plotting/bokeh/__init__.py
@@ -101,18 +101,20 @@
     pass
 
 
+point_size = np.sqrt(6) # Matches matplotlib default
 Cycle.default_cycles['default_colors'] =  ['#30a2da', '#fc4f30', '#e5ae38',
                                            '#6d904f', '#8b8b8b']
 
 options = Store.options(backend='bokeh')
 
 # Charts
 options.Curve = Options('style', color=Cycle(), line_width=2)
-options.Scatter = Options('style', color=Cycle(), size=np.sqrt(6))
+options.Scatter = Options('style', color=Cycle(), size=point_size)
+options.Points = Options('style', color=Cycle(), size=point_size)
 options.ErrorBars = Options('style', color='black')
 options.Spread = Options('style', fill_color=Cycle(), fill_alpha=0.6, line_color='black')
 options.Histogram = Options('style', fill_color="#036564", line_color="#033649")
-options.Points = Options('style', color=Cycle(), size=np.sqrt(6))
+
 options.Spikes = Options('style', color='black')
 options.Area = Options('style', color=Cycle(), line_color='black')
 

diff --git a/holoviews/plotting/bokeh/util.py b/holoviews/plotting/bokeh/util.py
@@ -186,6 +186,8 @@ def update_plot(old, new):
     Updates an existing plot or figure with a new plot,
     useful for bokeh charts and mpl conversions, which do
     not allow updating an existing plot easily.
+
+    ALERT: Should be replaced once bokeh supports it directly
     """
     old_renderers = old.select(type=GlyphRenderer)
     new_renderers = new.select(type=GlyphRenderer)