Automargin regression since 1.44.2 ?

[nicolaskruchten]

See this pen: https://codepen.io/nicolaskruchten/pen/yZrdKO?editors=0010

It takes a long time to run (almost hangs in Chrome/OSX!) and then the solution is to add a bunch of padding at the bottom, which didn't happen on 1.44.2, as in this pen: https://codepen.io/nicolaskruchten/pen/QYPeWE

[nicolaskruchten]

Maybe related to #3510 ?

[etpinard]

Why does xaxis2 have a title in your codepen?

Is this the desired behaviour: https://codepen.io/etpinard/pen/qgGxaW?editors=0010

[nicolaskruchten]

Why does xaxis2 have a title in your codepen?

it doesn't need one.

Is this the desired behaviour: https://codepen.io/etpinard/pen/qgGxaW?editors=0010

Yes. But if you're using a template which sets automargins (which plotly.py and Chart Studio do by default) then you get automargin: true on all axes by default. Setting automargins :true on both here even without the title makes the plot short again.

[etpinard]

Cool. On it!

[destradafilm]

See this pen: https://codepen.io/nicolaskruchten/pen/yZrdKO?editors=0010

It takes a long time to run (almost hangs in Chrome/OSX!) and then the solution is to add a bunch of padding at the bottom, which didn't happen on 1.44.2, as in this pen: https://codepen.io/nicolaskruchten/pen/QYPeWE

I'm seeing this same bug with automargin and I'm on v1.46.0, I even tested in v1.47 and still get it too. I think this hasn't been fixed.

[etpinard]

I'm seeing this same bug

By this same bug, you mean the page hangs?

[destradafilm]

yeah, I've been trying to recreate it but seems to need the perfect conditions to break. Even the original codepen submitted by @nicolaskruchten breaks sometimes for me.

can you test this one? it has a forced width / height (in pixels) to what the width / height in the original codepen (in vh) was rendered to on my browser.

https://codepen.io/destrada/pen/PgBroN?editors=1010

Eventually I get
plotly.min.js:1 Uncaught RangeError: Maximum call stack size exceeded

Although it doesn't happen to everyone. One colleague of mine can reproduce it with the same codepen, another reproduces it with this one
https://codepen.io/destrada/pen/yrqdgj?editors=1011

[etpinard]

OK thanks @destradafilm - I can't reproduce the problem from the two codepens you provided, but let me re-open the issue.

Can @plotly/plotly_js give:

• https://codepen.io/destrada/pen/yrqdgj?editors=1011
• https://codepen.io/destrada/pen/PgBroN?editors=1010

a shot and see if a max call-stack errors pops up?

[kevinoe]

I'm @destradafilm 's coworker who can also reproduce it with
https://codepen.io/anon/pen/LvBooa?editors=0011

It seems to be very sensitive to geometry. You might try playing with pixel sizes in that pen. From what I can tell, plotly calculates automargins, replots, which then results in another call to calculate automargins, which gets slightly different results (ie a value switches from xx.4x to xx.5x and gets rounded differently), which replots, etc etc.

I put a conditional breakpoint in doAutoMargin below these lines:

    if(!fullLayout._replotting &&
        oldmargins !== '{}' &&
        oldmargins !== JSON.stringify(fullLayout._size)
    ) {
        if('_redrawFromAutoMarginCount' in fullLayout) {`

with the condition that ` fullLayout._redrawFromAutoMarginCount > 100

The callstack I see is :

plots.doAutoMargin (plots.js:1927)
lib.syncOrAsync (index.js:401)
exports.plot (plot_api.js:396)
exports.call (registry.js:224)
plots.doAutoMargin (plots.js:1927)
lib.syncOrAsync (index.js:401)
exports.plot (plot_api.js:396)
exports.newPlot (plot_api.js:666)
(anonymous) (pen.js:3)

Where the last (top) 4 lines recurse until it fails

[kevinoe]

@etpinard - What sort of machine / os / browser are you using? We've seen a lot of variability in what triggers it.

For what its worth, I'm using:
MacBook Pro (13-inch, 2018, Four Thunderbolt 3 Ports)
OSX Mohave 10.14.4
Chrome Version 73.0.3683.103 (Official Build) (64-bit)

We have a fair variety of machines here - we might be able to try to tweak the code pen. Alternatively, I've created a new codepen (https://codepen.io/anon/pen/JVmPqB?editors=0011) which walks through a whole bunch of heights around the size I use. If you set the breakpoint I mentioned above, maybe you'll be able to reproduce