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

Not all series show is panel is too small #1730

Open
ethanalvizo opened this issue Jan 16, 2024 · 1 comment
Open

Not all series show is panel is too small #1730

ethanalvizo opened this issue Jan 16, 2024 · 1 comment
Labels
bug Something isn't working
Milestone

Comments

@ethanalvizo
Copy link
Contributor

Description
While working on #1654 I noticed that not all series show on the plot if the width of the panel is too small. Hovering over where the series data should be plotted still shows the data as being there, the line just doesn't show. Increasing the viewport width fixes the issue.

Steps to reproduce

  1. Same steps as Chart plotBy does not wait for all series to load before displaying #1654, enter the code below into the console
from deephaven import empty_table
from deephaven.plot.figure import Figure

t = empty_table(80000).update(["Timestamp=epochMillisToInstant(i)","x=(double)i", "y=i%12==11?null: i%5==0 ? i*2 : i", "series= i%5==0 ? `Double` : `Single`"])
f = Figure().plot_xy(series_name="Hello", t=t, x="Timestamp", y="y", by=["series"]).show()
  1. Resize the panel until one of the series disappears

Expected results
Both series are plotted on the chart
image

Actual results
The 'Hello: Single' series does not show on the plot but hovering over where the plot should be still correctly shows the data preview/tooltip
image

Additional details and attachments

Versions

Engine Version: 0.32.0-SNAPSHOT
Web UI Version: 0.58.0
Java Version: 11.0.19
Barrage Version: 0.6.0

@ethanalvizo ethanalvizo added bug Something isn't working triage Issue requires triage labels Jan 16, 2024
@vbabich
Copy link
Collaborator

vbabich commented Jan 23, 2024

There is a chance this related to #1046
We should retest this after updating plotly to 2.28

@vbabich vbabich removed the triage Issue requires triage label Jan 23, 2024
@vbabich vbabich added this to the Backlog milestone Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants