-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Can't find a pure Python 3 wheel for: 'tornado>=5.1' #2291
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
Comments
There is no tornado available in Pyodide https://pyodide.org/en/stable/usage/packages-in-pyodide.html |
That makes sense, but the Bokeh example used to work in the older version of PyScript. If you look at my code, all I did was take the Bokeh example from the PyScript website and update PyScript to the latest the version. |
is Bokeh using latest 3.6.0 version or it's pinned to an old one? 🤔 |
You will need to find the version of bokeh that was used in the example. Looking at the release history: Maybe "bokeh==3.6.0" is worth a try |
If I look at one of the bokeh examples:
|
While probably being an upstream dependency issue (from the looks of it, bokeh doesn't need tornado unless you are using the server), I think this issue is actually an opportunity to fix the user experience. Iirc, Bokeh is not the only data viz package that depends on Tornado for features that are not relevant in wasm based python interpreters. Adding a plugin or something like that, that installs and patches dependencies to make things work transparently would benefit the user experience. I don't think this change should be done at the pyscript codebase level though, or at least we should put some thought into it... |
all I can think about is a |
ok ... after a quick search I couldn't find any way to tell micropip that tornado is not really needed but I wonder if adding the ability to mock packages could help solving those one missing or conflicting ... https://micropip.pyodide.org/en/v0.7.1/project/api.html#micropip.add_mock_package I still think not pinning packages is still "the elephant in the room" and we should stress all over the docs that issues like this one will keep happening every single time a package introduces a dependency that is not available, or changes its API in a breaking way. |
Checklist
What happened?
If I update the Bokeh example to use the current version of PyScript, it fails with this error.
https://pyscript.com/@08c3cff3-0e42-4fe0-91bb-626eef1ce818/bokeh-copy-copy/latest?files=README.md,main.py,index.html
What browsers are you seeing the problem on? (if applicable)
Chrome
Console info
Additional Context
No response
The text was updated successfully, but these errors were encountered: