[BUG]: Mixed bar orientations hide one bar on initial render, but restyle renders correctly
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 45/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- javascript, playwright, python
- Domaine
- data-visualization, frontend
Piste de recherche
Exécutez la reproduction fournie en Python et Playwright, puis comparez la capture d’écran du rendu initial avec le résultat de Plotly.restyle. Suivez le chemin de rendu des barres pour un mélange d’orientations horizontales et verticales ; le travail est terminé lorsque les deux rendus affichent les trois barres rouges sans rognage ni différences causées par le timing du rendu.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Description
A figure with two go.Bar traces renders differently depending on when the second
trace is changed from horizontal to vertical.
If fig.data[1].orientation = "v" is set before the first render, one red vertical
bar is missing or clipped. If the same base figure is rendered first and the same
orientation value is applied later with Plotly.restyle, all three red bars are
visible.
Screenshots/Video
Steps to reproduce
import io
import plotly
import plotly.graph_objects as go
import plotly.io as pio
from PIL import Image, ImageChops
from playwright.sync_api import sync_playwright
print(f"Plotly version: {plotly.__version__}")
fig = go.Figure(
[
go.Bar(name="a", orientation="h", x=[1, 3, 2], y=[0, 1, 2]),
go.Bar(name="b", orientation="h", x=[3, 2, 1], y=[0, 1, 2]),
]
)
fig.update_layout(xaxis_title="value", yaxis_title="index", legend_title_text="variable")
# plot1.png: set orientation before the first render.
fig.data[1].orientation = "v"
assert fig.data[1].orientation == "v"
html1 = pio.to_html(fig, full_html=True, include_plotlyjs="cdn")
# plot2.png: render first, then apply the same value with Plotly.restyle.
fig.data[1].orientation = "h"
html2 = pio.to_html(fig, full_html=True, include_plotlyjs="cdn")
html2 = html2.replace(
"</body>",
"""
<script>
window.addEventListener("DOMContentLoaded", () => {
const plotEl = document.getElementsByClassName("plotly-graph-div")[0];
setTimeout(() => {
Plotly.restyle(plotEl, {"orientation": "v"}, [1]);
}, 1000);
});
</script>
</body>""",
)
with sync_playwright() as playwright:
browser = playwright.chromium.launch(
headless=True,
args=["--no-sandbox", "--disable-dev-shm-usage", "--disable-gpu"],
)
page = browser.new_page(viewport={"width": 1200, "height": 800})
page.set_content(html1)
page.wait_for_timeout(1500)
img1 = page.screenshot(path="plot1.png")
page.set_content(html2)
page.wait_for_timeout(2000)
img2 = page.screenshot(path="plot2.png")
browser.close()
diff = ImageChops.difference(
Image.open(io.BytesIO(img1)).convert("RGB"),
Image.open(io.BytesIO(img2)).convert("RGB"),
)
assert diff.getbbox() is not None, "Screenshots are unexpectedly identical."
print("Saved plot1.png and plot2.png")
```### Notes
_Add info here that doesn't fit in the other sections._
- Langage dominant
- Python
- Étoiles
- 18.8k
- Forks
- 2.8k
- Merge moyen
- 13 h 28 min
- PR mergées (30 j)
- 20
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de plotly/plotly.py
-
P3 size: 1 task
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
P3 size: 1 task
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
bug P1
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
-
feature P3
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
-
feature P3
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Toutes les issues de plotly/plotly.py
Issues similaires
-
agent-ready documentation needs-triage
Difficulté 1/5 1-3 heures Accessibilité débutants 88/100
-
documentation
Difficulté 1/5 Moins d'une heure Accessibilité débutants 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 92/100
-
Add https://search.jeremyh.xyz/ Ouverteinstance instance add
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
searxng/searx-instances#939 · 1 commentaire ·
-
area-deployment area-integrations triage:bot-seen
Difficulté 2/5 Une demi-journée Accessibilité débutants 86/100