Skip to content

Conversation

@tconkling
Copy link
Contributor

@tconkling tconkling commented Apr 7, 2021

  • bokehjs: 2.0.0 -> 2.2.2
  • ran npx browserslist@latest --update-db

This is not a solution for #2156. The latest bokehjs is 2.3.1, but using bokehjs 2.3.0 or 2.3.1 gives a runtime error when the chart is rendered:

"ReferenceError: Cannot access ‘ColumnarDataSource’ before initialization".

It looks like this is a cyclic dependency issue, and that it's been addressed by bokehjs here: bokeh/bokeh#11140. Presumably, when bokehjs 2.4 releases, we'll be able to it.

(More context in the bokehjs forums, here: https://discourse.bokeh.org/t/problem-importing-bokeh-bokehjs-module-in-javascript-app/7510/12)

@tconkling tconkling requested a review from a team April 7, 2021 22:37
@tconkling tconkling merged commit cd97d1c into streamlit:develop Apr 7, 2021
@tconkling tconkling deleted the tim/UpgradeBokehJS branch April 7, 2021 23:08
tconkling added a commit that referenced this pull request Apr 7, 2021
* develop:
  Upgrade bokehjs to 2.2.2, and run `npx browserslist@latest --update-db` (#3090)
  remove extra star in docs (#3086)
  Remove flake8 linting tool (#3085)
tconkling added a commit that referenced this pull request Apr 13, 2021
* develop: (165 commits)
  Add s4a message to close active modals (#2893)
  Add fail_on_warning to build options
  Docutils 0.16
  Put back Sphinx to 3.0.3
  Add import urllib to streamlit hello (#2969) (#3106)
  Update Num_Input to correct for Type Errors (#3074)
  Upgrade bokehjs to 2.2.2, and run `npx browserslist@latest --update-db` (#3090)
  remove extra star in docs (#3086)
  Remove flake8 linting tool (#3085)
  Makefile cleanup (#3083)
  Merge anchor headers feature branch into develop (#2983)
  Fix image galleries (#3044)
  Is this a red herring also?
  See if pylint was red herring
  Update sphinx
  Specify pylint version
  Fix pylint errors
  Added "allow-downloads" to the sandbox attributes (#3053)
  Small fix for `make pylint` command (#3062)
  Set genericColors properly and make theme defs more consistent (#3051)
  ...
kmcgrady pushed a commit to kmcgrady/streamlit that referenced this pull request May 3, 2021
kmcgrady pushed a commit that referenced this pull request May 4, 2021
kmcgrady pushed a commit that referenced this pull request May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants