Add ipywidgets dependency to tutorial notebooks - #213
Conversation
The cosyne_2023 and neurodatarehack_2024 dandiset-search notebooks import `tqdm.notebook`, whose progress bar needs ipywidgets. Their install cells lost the ipywidgets pin when the locks were regenerated from requirements.in, which listed only dandi and tqdm. Colab preinstalls ipywidgets so the notebooks still ran there, but CI installs only the pinned set, and the weekly sweep failed with "IProgress not found". Declare ipywidgets as a direct dependency and re-lock. The new pins are additions only, at Colab's versions (ipywidgets 7.7.1 and its notebook stack) apart from jedi, which Colab does not ship. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PntFeR57FsYaeMEk4XGoii
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
Preview for this PR has been removed (PR closed). |
Executed notebooksCI executes the notebooks changed in this PR through a Jupyter kernel and publishes each one, with its outputs, as soon as it finishes. Links appear below as notebooks complete; an hourglass means the notebook is still running.
3 of 3 finished. Commit |
|
@bendichter are you getting those emails about failing CI for this repo which seems never were green? https://github.com/dandi/example-notebooks/actions/workflows/test-all-notebooks-weekly.yml I think this might make it green finally! Let's do it?! |
4 participants
Summary
This PR adds
ipywidgetsas an explicit dependency to the tutorial notebooks that require interactive widget functionality.Changes
ipywidgetsto the requirements for three tutorial notebooks:tutorials/cosyne_2023/simple_dandiset_search.ipynbtutorials/neurodatarehack_2024/advanced_asset_search.ipynbtutorials/neurodatarehack_2024/simple_dandiset_search.ipynb.requirements.infiles to includeipywidgetsas an explicit dependencyipywidgetsand its transitive dependenciesDetails
The addition of
ipywidgetsensures that interactive widgets are available in the tutorial environments. This change was made to the source.requirements.infiles, and the locked dependency files were regenerated to capture all transitive dependencies needed for proper functionality.https://claude.ai/code/session_01PntFeR57FsYaeMEk4XGoii