Skip to content

Commit

Permalink
Fix bins option in autompg_histogram demo (#5750)
Browse files Browse the repository at this point in the history
  • Loading branch information
TheoMathurin authored and hoxbro committed Jun 11, 2023
1 parent dfb2a96 commit 5649eb7
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
2 changes: 1 addition & 1 deletion examples/gallery/demos/bokeh/autompg_histogram.ipynb
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
"metadata": {},
"outputs": [],
"source": [
"hist = autompg_ds.hist(dimension='mpg', groupby='cyl', bin_range=(9, 46), bins=40, adjoin=False)\n",
"hist = autompg_ds.hist(dimension='mpg', groupby='cyl', bin_range=(9, 46), num_bins=40, adjoin=False)\n",
"\n",
"hist.opts(opts.Histogram(alpha=0.9, width=600))"
]
Expand Down
7 changes: 6 additions & 1 deletion holoviews/plotting/bokeh/chart.py
Original file line number Diff line number Diff line change
Expand Up @@ -797,7 +797,12 @@ def _axis_properties(self, axis, key, plot, dimension=None,
((not self.invert_axes and axis == 'x') or (self.invert_axes and axis =='y'))):
props['separator_line_width'] = 0
props['major_tick_line_alpha'] = 0
props['major_label_text_font_size'] = '0pt'
# The major_label_text_* is a workaround for 0pt font size not working in Safari.
# See: https://github.com/holoviz/holoviews/issues/5672
props['major_label_text_font_size'] = '1px'
props['major_label_text_alpha'] = 0
props['major_label_text_line_height'] = 0

props['group_text_color'] = 'black'
props['group_text_font_style'] = "normal"
if axis == 'x':
Expand Down

0 comments on commit 5649eb7

Please sign in to comment.