Skip to content

Conversation

@fleming79
Copy link

@fleming79 fleming79 commented Apr 30, 2025

Resolves: #488

Requires jupyter-widgets/ipywidgets#4002 (or similar)

@github-actions
Copy link
Contributor

Binder 👈 Launch a binder notebook on branch fleming79/ipympl/main

@fleming79 fleming79 reopened this Jan 19, 2026
@fleming79
Copy link
Author

fleming79 commented Jan 19, 2026

@ianhi - thank you for starting the workflow.

I'd accidentally made the proposed changes to ipympl using a patched version of Ipywidgets PR.

The patched Ipywidgets version creates a comm when the model_id is accessed detail here.

I've temporarily changed the dependency to install the patched ipympl to see if the tests will pass.

Would you mind re-running the workflows?

@fleming79 fleming79 marked this pull request as draft January 19, 2026 22:36
@fleming79 fleming79 force-pushed the main branch 2 times, most recently from 00e8a1a to 4f9ad94 Compare January 20, 2026 03:20
@fleming79
Copy link
Author

@ianthomas23 - thank you for running the workflow. Unfortunately, I don't think the patched version was used in the test even though it was downloaded. I've changed it to force-install the patch just prior to running the test in the workflow. Could you please trigger the workflow again?

@fleming79
Copy link
Author

Thanks @ianthomas23 - The patched version of Ipywidgets requires Python >= 3.10 which explains the failing tests.

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.

Warnings with backend_nbagg

1 participant