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

Scatter plots get bunched up and plot at the wrong spots on M1/M3 macs #323

Closed
mofojed opened this issue Feb 28, 2024 · 0 comments · Fixed by #324
Closed

Scatter plots get bunched up and plot at the wrong spots on M1/M3 macs #323

mofojed opened this issue Feb 28, 2024 · 0 comments · Fixed by #324
Labels
bug Something isn't working triage

Comments

@mofojed
Copy link
Member

mofojed commented Feb 28, 2024

Description

When you plot a scatter plot against time on a M1 or M3 Mac, Plotly does not render the results correctly.
Similar to deephaven/web-client-ui#1046. Should be fixed by Plotly in 2.28: plotly/plotly.js#5970 (comment)

Steps to reproduce

  1. Run the following snippet and let the chart tick for a bit:
from deephaven import time_table
from deephaven.plot import express as dx

trades = time_table('PT0.1S').update(['ts=Timestamp', 'price=i', 'exchange=`EX`'])

p = dx.scatter(
    trades,
    x="ts", 
    y=["price"],
    color_discrete_sequence=["red", "lightgreen", "lightblue"],
    size_sequence=5,
    title="Trades (last 10 minutes)",
    xaxis_titles = '',
    yaxis_titles = 'Price ($)',
    by=["exchange"], 
)

Expected results

  1. Scatter plot should be distributed according to the data

Actual results

  1. Scatter plot appears, but x values are all bunched together. See screenshot

Additional details and attachments
image

Versions

Engine Version: 0.34.0-SNAPSHOT
Web UI Version: 0.66.0
Java Version: 11.0.21
Barrage Version: 0.6.0
Browser Name: Chrome 121
OS Name: macOS 10.15.7

@mofojed mofojed added bug Something isn't working triage labels Feb 28, 2024
mofojed added a commit to mofojed/deephaven-plugins that referenced this issue Feb 28, 2024
- Update plotly.js to fix the issue
- Fixes deephaven#323
- Verified using the steps in the issue on my M1 mac (issue was reproducible before this change)
mofojed added a commit that referenced this issue Feb 28, 2024
- Update plotly.js to fix the issue
- Fixes #323
- Verified using the steps in the issue on my M1 mac (issue was
reproducible before this change)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant