Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plotly - multiple series plots with color axis cause error when trying to move panel #40

Open
mofojed opened this issue Jun 6, 2023 · 2 comments
Labels
bug Something isn't working Javascript Involves Javascript side of a plugin plotly-express Involves plotly-express plugin

Comments

@mofojed
Copy link
Member

mofojed commented Jun 6, 2023

Description

Steps to reproduce

  1. Create a plotly figure with multiple series and plot types:
from plotly.subplots import make_subplots
import plotly.graph_objects as go
fig = make_subplots(rows=1, cols=2)
fig.add_trace(
    go.Scatter(x=[1, 2, 3], y=[4, 5, 6]),
    row=1, col=1)
fig.add_trace(
    go.Bar(x=[1, 2, 3], y=[4, 5, 6], marker=dict(color=[4, 5, 6], coloraxis="coloraxis")),
    row=1, col=2)
fig.update_layout(title_text="Side By Side Subplots", showlegend=False)
  1. Click and drag the figure to place it somewhere else in the layout

Expected results
2. Plot should move

Actual results
2. Plot gets stuck, layout is broken:
image

@mofojed mofojed added the bug Something isn't working label Jun 6, 2023
@mofojed mofojed changed the title Plotly - multiple series plots cause error when trying to move panel Plotly - multiple series plots with color axis cause error when trying to move panel Jun 6, 2023
@mofojed
Copy link
Member Author

mofojed commented Jun 6, 2023

Note - if I remove coloraxis="coloraxis" it seems to work okay.

@mofojed
Copy link
Member Author

mofojed commented Jun 6, 2023

I've filed an issue in plotly.js for this: plotly/plotly.js#6630

I've created a codepen that narrows the issue down to a coloraxis and autosizing: https://codepen.io/mofojed/pen/WNYNpBv

mofojed referenced this issue in mofojed/web-client-ui Jun 7, 2023
- Added a try/catch around all events that are emitted from Golden Layout
- Added @deephaven/log to golden layout so we can log the error
- Tested using the steps in https://github.com/deephaven/deephaven-js-plugins/issues/26
- Fixes deephaven#1352
mofojed referenced this issue in mofojed/web-client-ui Jun 8, 2023
- Added a try/catch around all events that are emitted from Golden Layout
- Added @deephaven/log to golden layout so we can log the error
- Tested using the steps in https://github.com/deephaven/deephaven-js-plugins/issues/26
- Fixes deephaven#1352
mofojed referenced this issue in deephaven/web-client-ui Jun 8, 2023
…out (#1353)

- Added a try/catch around all events that are emitted from Golden
Layout
- Added @deephaven/log to golden layout so we can log the error
- Tested using the steps in
https://github.com/deephaven/deephaven-js-plugins/issues/26
- Fixes #1352
@jnumainville jnumainville transferred this issue from deephaven/deephaven-js-plugins Sep 12, 2023
@jnumainville jnumainville added Javascript Involves Javascript side of a plugin plotly-express Involves plotly-express plugin labels Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Javascript Involves Javascript side of a plugin plotly-express Involves plotly-express plugin
Projects
None yet
Development

No branches or pull requests

2 participants