Skip to content

fix: plotly regression - #8471

Merged
hoxbro merged 1 commit into
mainfrom
fix_plotly_regression
Mar 5, 2026
Merged

fix: plotly regression#8471
hoxbro merged 1 commit into
mainfrom
fix_plotly_regression

Conversation

@hoxbro

@hoxbro hoxbro commented Mar 5, 2026

Copy link
Copy Markdown
Member

Description

Fixes #8470

How Has This Been Tested?

Running the original issue code locally and CI.

AI Disclosure

  • This PR contains AI-generated content.
    • I have tested all AI-generated content in my PR.
    • I take responsibility for all AI-generated content in my PR.

Tools: Claude code for making an MRE and adding the UI test.

Checklist

  • Tests added and is passing

@codecov

codecov Bot commented Mar 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.55172% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 86.18%. Comparing base (fb924e0) to head (8dfce2a).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
panel/tests/ui/pane/test_plotly.py 94.73% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #8471       +/-   ##
===========================================
+ Coverage   70.44%   86.18%   +15.74%     
===========================================
  Files         348      349        +1     
  Lines       55025    55048       +23     
===========================================
+ Hits        38760    47442     +8682     
+ Misses      16265     7606     -8659     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hoxbro
hoxbro force-pushed the fix_plotly_regression branch from 5552afc to 8dfce2a Compare March 5, 2026 15:43
@hoxbro
hoxbro marked this pull request as ready for review March 5, 2026 15:55
@hoxbro
hoxbro merged commit 584416e into main Mar 5, 2026
19 of 20 checks passed
@hoxbro
hoxbro deleted the fix_plotly_regression branch March 5, 2026 15:59
@hoxbro hoxbro mentioned this pull request Mar 6, 2026
1 task
@github-actions

Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Plotly callback broken in >=v1.8.7

1 participant