Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: capture stderr/stdout in output widget #132

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

feat: capture stderr/stdout in output widget #132

wants to merge 1 commit into from

Conversation

maartenbreddels
Copy link
Contributor

@maartenbreddels maartenbreddels commented Jun 1, 2023 •
edited
Loading

Fixes #127

TODO:

  • hook support for clear_output should also go upstream to ipykernel
  • tests
  • make sure the whole API surface is covered for proper stdout/stderr support
@maartenbreddels maartenbreddels mentioned this pull request Jun 1, 2023
Open
maartenbreddels added a commit to maartenbreddels/ipykernel that referenced this pull request Jul 20, 2023
@maartenbreddels
feat: let display hook handle clear_output
e4ede7e 
A display hook can handle a publish message, but not yet a clear_output

Upstreaming of: widgetti/solara#132
Follow up of: ipython#1110
Related: https://github.com/ipython/ipykernel/pull/115/files

Would enable jupyter-widgets/ipywidgets#3759
to fully work kernel side.
@maartenbreddels maartenbreddels mentioned this pull request Jul 20, 2023
Merged
@maartenbreddels maartenbreddels changed the title feat: capture stderr/stdout in output widget and support clear_output Jul 24, 2023
@iisakkirotko iisakkirotko force-pushed the master branch 2 times, most recently from c2951b9 to 8816e36 Compare April 5, 2024 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant
@maartenbreddels