autorange should not display zero line for certain bars with positive base and negative value

[emmanuelle]

See for example https://codepen.io/emmanuelle-plotly/pen/bGdOLmR?editable=true, the axis starts at 0 and therefore a large part of the domain is not used to represent data. For other sets of data the automatic range works well.

Triggered by plotly/plotly.py#2327

[archmoj]

Something possibly went wrong in autorange when using base and the sum (without base) goes below the initial value.

[archmoj]

Minimal demo.

[archmoj]

It works well if we force tozero to be false e.g. here:

plotly.js/src/plots/cartesian/autorange.js

Line 335 in ebab12b

var tozero = opts.tozero && (ax.type === 'linear' || ax.type === '-');

[archmoj]

Not sure if we should have tag this as a bug.
Demo using bar.

[alexcjohnson]

Definitely a bug - and we have logic that appears intended to address exactly this, but somehow isn't working.

plotly.js/src/traces/bar/cross_trace_calc.js

Line 521 in b8a452c

tozero: !allBaseAboveZero,

[archmoj]

minimal demo in regards with the new title.

[alexcjohnson]

Ah, nice sleuthing @archmoj! In fact, to dive into the details of tozero a bit: if an axis is autoranged tozero it means two things: (1) zero is always included in the autorange, and (2) if the lowest (or highest) point on the axis is exactly zero, no padding is added past zero. This is an interesting effect: it says that for a normal bar chart or filled area chart, the size axis starts exactly at zero, but we get the normal 5% padding at the other end.

I wonder if the correct tozero logic is as simple as "true if any bar has zero base"? I don't think it matters whether the base is positive or negative, or anything about the bar size.

[archmoj]

@alexcjohnson I think we still want to display the zero line when the values are negative.
So this codepen is OK.

[archmoj]

See for example https://codepen.io/emmanuelle-plotly/pen/bGdOLmR?editable=true, the axis starts at 0 and therefore a large part of the domain is not used to represent data. For other sets of data the automatic range works well.

@emmanuelle thanks very much for the report. In your codepen, would you mind set plotly.js version to 1.52.3 and not use the latest?

[alexcjohnson]

I think we still want to display the zero line when the values are negative.
So this codepen is OK.

Why? What makes that case different from its positive twin?

[archmoj]

I don't have strong opinion on that. Perhaps as a result of this bug, I always thought we wanted to display zerolines for bars with negative values.