-
Notifications
You must be signed in to change notification settings - Fork 74
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
Python: Add support for additional widget types (in viewer or plots pane) #1924
Comments
@isabelizimm I imagine these were all tested prior to the switchover to the later version of html-manager, it might be worth us re-testing to see whether there are any changes in what works/what doesn't after the update -- we should also specify here the module version of the above widget types that we tested, since we know now that makes a difference to whether it's compatible or not |
In order to support matplotlib widgets and others that require a live Python kernel, we need to enable an actual non-stubbed See current TODO in positron/src/vs/workbench/contrib/positronOutputWebview/browser/notebookOutputWebviewServiceImpl.ts Lines 189 to 193 in aa6476a
and stubbed RendererContext positron/src/vs/workbench/contrib/positronOutputWebview/browser/notebookOutputWebviewServiceImpl.ts Lines 208 to 219 in aa6476a
Also see how this gets handled for ipywidgets in the jupyter extension |
Does this cover plotly / bokeh plots? |
@peekxc I believe that plotly works: Are you observing something different? For bokeh, I do not believe we have that working correctly right now (the plot opens in an external browser). |
#4172 and #4208 should get the following widgets working:
There may be smaller issues with those, which we can log separately. I haven't yet figured out why |
Verified Fixed
Test scenario(s)Validated with everything here: https://gist.github.com/isabelizimm/02f7ba16dc74b202935f6d3e9ed6b892 Link(s) to TestRail test cases run or created: |
Once #1918 is merged, we will have basic support for the widgets such as ipywidgets, ipydatagrid, and ipyleaflet in the plot pane. Still not working ❌ is:
Here is a gist of all the different widgets, for testing purposes
CC: @jgutman
The text was updated successfully, but these errors were encountered: