Skip to content

Conversation

@ascibisz
Copy link
Contributor

@ascibisz ascibisz commented Dec 17, 2024

Time Estimate or Size

Medium

Problem

Previously, the nbsv widget only works in Jupyterlab, but it would be ideal if it works in Google Collab (and Jupyter Notebook while we're at it)
Link to story or ticket

Solution

After fumbling with debugging for a long time, I discovered the anywidget package which is meant to handle a lot of the platform specific configurations, allowing widgets that use it to work on multiple different platforms. This allowed me to get rid of some of the weird configurations I was trying to use to get rid of errors in google colab!

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file contains what used to be in src/widget.ts, with some changes to use anywidget

@ascibisz ascibisz marked this pull request as ready for review January 7, 2025 16:43
@ascibisz ascibisz requested a review from a team as a code owner January 7, 2025 16:43
@ascibisz ascibisz requested review from frasercl, interim17 and meganrm and removed request for a team January 7, 2025 16:43
Copy link
Contributor

@interim17 interim17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome that we can run in jupyter notebook as well as colab now

@ascibisz ascibisz merged commit 8a7e844 into main Jan 10, 2025
6 checks passed
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.

4 participants