scatter_matrix formatting considered harmful about hvplot HOT 9 OPEN

Just adding that the spacing issue seems unrelated to the toolbar:

scatter_matrix(df, alpha=0.5, width=600, height=600, xrotation=0).opts(toolbar='above')

from hvplot.

Just adding that the spacing issue seems unrelated to the toolbar:

Yes, I think the spacing issues have been there for some time, while the toolbar issue is relatively recent, but I haven't tried to do a git bisect to pin that down.

from hvplot.

Quick feedback:

• Toolbar issue: possibly introduced in holoviz/holoviews#5873
• Subplots not aligned: happened sometime around the Bokeh 3 transition (not sure it's a Bokeh 3 problem either)

https://holoviews.org/reference/containers/bokeh/GridSpace.html

There may be no hvPlot issue at all.

from hvplot.

Thanks. I've opened holoviz/holoviews#6126 for the toolbar issue, and @mattpap is looking at it from the Bokeh side.

from hvplot.

Bad plot alignment is caused by fixed frame sizing (Plot.frame_{width,height,align}), which works reliably only for single plots and doesn't work well in all other cases (see e.g. issue bokeh/bokeh#13225). I suppose it's time to implement this properly.

from hvplot.

From the initial list of issues:

1. subplots all lined up: Mateusz indicated this is a Bokeh issue
2. a single Bokeh toolbar for the entire figure: HoloViews issue fixed in holoviz/holoviews#6127
3. axis labels readable, text not overlapping

That leaves us with 3). The default Bokeh formatter is the BasicTickFormatter:

Comparing that to the default of plotly express:

We can get a similar behavior defining a NumericalTickFormatter:

However, it also has its limits:

Certainly, we could better document xformatter/yformatter. But should we also consider defaulting to a more user-friendly formatter?

from hvplot.

Defaulting to a more usable formatter sounds like a great idea. @mattpap , any idea why the tick formatter didn't decide to drop the intermediate tick marks? Here I'd be hoping to get one label on the left of the x axis, and one on the right:

from hvplot.

any idea why the tick formatter didn't decide to drop the intermediate tick marks?

This is handled setting Axis.major_label_policy = NoOverlap(). When this was implemented the default (AllLabels) was left for backwards compatibility. Tickers and tick formatters have no access to the screen space, so they can't make any adjustments based on the positioning of labels.

from hvplot.

Thanks! Ok, @maxime, can you try out the NoOverlap option with NumericalTickFormatter? For hvPlot I strongly favor improving the user experience over preserving previous defaults.

from hvplot.