Remove native bokeh charts support by lukasmasuch · Pull Request #12964 · streamlit/streamlit

lukasmasuch · 2025-11-07T00:23:43Z

Describe your changes

Removes the native st.bokeh_chart implementation and changes the command to be a no-op call that just shows a message redirecting users to use https://github.com/streamlit/streamlit-bokeh

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

snyk-io · 2025-11-07T00:23:52Z

✅ Snyk checks have passed. No issues have been found so far.

Status	Scanner	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues
✅	Licenses	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

github-actions · 2025-11-07T00:24:00Z

✅ PR preview is ready!

Name	Link
📦 Wheel file	https://core-previews.s3-us-west-2.amazonaws.com/pr-12964/streamlit-1.51.0-py3-none-any.whl
📦 `@streamlit/component-v2-lib`	Download from artifacts
🕹️ Preview app	pr-12964.streamlit.app (☁️ Deploy here if not accessible)

cursor · 2025-11-07T00:31:54Z

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on December 3.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

github-actions · 2025-11-07T00:38:40Z

📉 Significant wheel size change detected

The wheel file size has decreased by 10.33% (threshold: 0.25%)

Current PR: 8865.05 KB
Latest develop: 9886.13 KB

Please verify this change is expected.

sfc-gh-lmasuch · 2025-11-20T16:59:01Z

e2e_playwright/basic_app_test.py

    # size of web assets low, you can modify this threshold if it's really needed
    # to add some new features. But make sure that its justified and intended.
-    TOTAL_ASSET_SIZE_THRESHOLD_MB: Final = 7.5  # noqa: N806
+    TOTAL_ASSET_SIZE_THRESHOLD_MB: Final = 7.9  # noqa: N806


It is not great that this change randomly causes an uptick in the initial bundle size. However, I don't think it is worth delaying this change. We will anyways work on optimizing our assets later this quarter, so this is expected to be resolved in ~1.53 / ~1.54.

Copilot

Pull Request Overview

This PR removes the native Bokeh chart support from Streamlit, converting st.bokeh_chart() into a no-op command that displays a deprecation warning directing users to the streamlit-bokeh custom component.

Key changes:

Backend: st.bokeh_chart() now only shows a deprecation warning and returns without rendering
Frontend: Complete removal of BokehChart component and all Bokeh vendor files
Dependencies: Removal of bokeh, numpy<2 requirements and react-debounce-render package
Tests: Simplified to only verify deprecation warning is shown

Reviewed Changes

Copilot reviewed 14 out of 28 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
`lib/streamlit/elements/bokeh_chart.py`	Converted to no-op with deprecation warning
`lib/tests/streamlit/elements/bokeh_test.py`	Removed all functional tests, kept deprecation test only
`lib/integration-requirements.txt`	Removed bokeh and numpy<2 dependencies
`mypy.ini`	Removed bokeh type ignore configuration
`frontend/lib/src/components/elements/BokehChart/*`	Deleted entire BokehChart component
`frontend/lib/src/vendor/bokeh/*`	Deleted all vendored Bokeh files
`frontend/lib/package.json`	Removed react-debounce-render dependency
`e2e_playwright/basic_app_test.py`	Increased asset size threshold from 7.5 to 7.9 MB
`NOTICES`	Removed bokeh and react-debounce-render license notices
`Makefile`	Removed bokeh license append step

e2e_playwright/basic_app_test.py

lib/streamlit/elements/bokeh_chart.py

sfc-gh-bnisco

🎉

sfc-gh-bnisco · 2025-11-20T19:39:48Z

lib/streamlit/elements/bokeh_chart.py

-
-            For more current updates, use the |streamlit-bokeh|_ custom
-            component instead.
+            This command has been deprecated and removed. Please use the


nit: Might be something for @sfc-gh-dmatthews to look at, but we should ensure that the wording for the docstring and show_deprecation_warning are aligned. One states "Please use the custom component" and the other states "Please use our custom component." Subtle but different language, and it might make sense to choose one or the other for consistency.

Updated to use the same wording 👍 Debbie will probably do additional tweaks before the release.

lukasmasuch · 2025-11-20T21:28:09Z

Waiting for #13078 before merging this in.

Remove native bokeh charts implementation

6711bdf

Update

a06f4b4

lukasmasuch added security-assessment-completed change:other PR contains other type of change impact:users PR changes affect end users labels Nov 7, 2025

Add type ignore

3fdfaca

lukasmasuch marked this pull request as ready for review November 7, 2025 00:31

lukasmasuch requested a review from a team as a code owner November 7, 2025 00:31

lukasmasuch changed the title ~~Remove native bokeh charts implementation~~ Remove native bokeh charts support Nov 7, 2025

lukasmasuch added the change:breaking PR contains breaking change that affects backwards compatibility label Nov 7, 2025

Improve

39d4376

lukasmasuch marked this pull request as draft November 8, 2025 00:08

lukasmasuch changed the title ~~Remove native bokeh charts support~~ [WIP] Remove native bokeh charts support Nov 8, 2025

lukasmasuch added 2 commits November 20, 2025 17:55

Merge remote-tracking branch 'upstream/develop' into remove-native-bokeh

93b407f

Use higher limit

44500bf

sfc-gh-lmasuch marked this pull request as ready for review November 20, 2025 16:55

sfc-gh-lmasuch reviewed Nov 20, 2025

View reviewed changes

sfc-gh-lmasuch changed the title ~~[WIP] Remove native bokeh charts support~~ Remove native bokeh charts support Nov 20, 2025

sfc-gh-lmasuch requested a review from Copilot November 20, 2025 17:00

Copilot started reviewing on behalf of sfc-gh-lmasuch November 20, 2025 17:00 View session

Copilot finished reviewing on behalf of sfc-gh-lmasuch November 20, 2025 17:01

Copilot AI reviewed Nov 20, 2025

View reviewed changes

e2e_playwright/basic_app_test.py Show resolved Hide resolved

lib/streamlit/elements/bokeh_chart.py Show resolved Hide resolved

lib/streamlit/elements/bokeh_chart.py Show resolved Hide resolved

sfc-gh-bnisco approved these changes Nov 20, 2025

View reviewed changes

Update docstring

e818813

Merge branch 'develop' into remove-native-bokeh

f1af4c6

Merge branch 'develop' into remove-native-bokeh

0da3f77

lukasmasuch enabled auto-merge (squash) November 21, 2025 09:51

lukasmasuch added 2 commits November 21, 2025 13:17

Update snapshot

3384a3e

Add additional timeout

c65f96f

lukasmasuch added the flaky-verify If applied to PR will run flaky test verification workflow label Nov 21, 2025

Update

951454a

lukasmasuch disabled auto-merge November 21, 2025 13:24

lukasmasuch added flaky-verify If applied to PR will run flaky test verification workflow and removed flaky-verify If applied to PR will run flaky test verification workflow labels Nov 21, 2025

Remove

29ef1b3

lukasmasuch enabled auto-merge (squash) November 21, 2025 13:36

lukasmasuch removed the flaky-verify If applied to PR will run flaky test verification workflow label Nov 21, 2025

lukasmasuch merged commit 6021727 into develop Nov 21, 2025
43 checks passed

lukasmasuch deleted the remove-native-bokeh branch November 21, 2025 14:00

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Remove native bokeh charts support#12964

Remove native bokeh charts support#12964
lukasmasuch merged 13 commits intodevelopfrom
remove-native-bokeh

lukasmasuch commented Nov 7, 2025 •

edited

Loading

Uh oh!

snyk-io bot commented Nov 7, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Nov 7, 2025 •

edited

Loading

Uh oh!

cursor bot commented Nov 7, 2025

Uh oh!

github-actions bot commented Nov 7, 2025 •

edited

Loading

Uh oh!

sfc-gh-lmasuch Nov 20, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sfc-gh-bnisco left a comment

Uh oh!

sfc-gh-bnisco Nov 20, 2025

Uh oh!

lukasmasuch Nov 20, 2025

Uh oh!

lukasmasuch commented Nov 20, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

lukasmasuch commented Nov 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Describe your changes

Uh oh!

snyk-io bot commented Nov 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Snyk checks have passed. No issues have been found so far.

Uh oh!

github-actions bot commented Nov 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ PR preview is ready!

Uh oh!

cursor bot commented Nov 7, 2025

Uh oh!

github-actions bot commented Nov 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📉 Significant wheel size change detected

Uh oh!

sfc-gh-lmasuch Nov 20, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sfc-gh-bnisco left a comment

Choose a reason for hiding this comment

Uh oh!

sfc-gh-bnisco Nov 20, 2025

Choose a reason for hiding this comment

Uh oh!

lukasmasuch Nov 20, 2025

Choose a reason for hiding this comment

Uh oh!

lukasmasuch commented Nov 20, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

lukasmasuch commented Nov 7, 2025 •

edited

Loading

snyk-io bot commented Nov 7, 2025 •

edited

Loading

github-actions bot commented Nov 7, 2025 •

edited

Loading

github-actions bot commented Nov 7, 2025 •

edited

Loading