Fix: missing 1 required positional argument: 'canvas' in Toolbar init#595
Fix: missing 1 required positional argument: 'canvas' in Toolbar init#595fleming79 wants to merge 3 commits intomatplotlib:mainfrom
Conversation
|
@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? |
00e8a1a to
4f9ad94
Compare
|
@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? |
|
Thanks @ianthomas23 - The patched version of Ipywidgets requires Python >= 3.10 which explains the failing tests. |
Resolves: #488
Requires jupyter-widgets/ipywidgets#4002 (or similar)