You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[Doc]: the matplotlib notebook magic requires notebook<7 #27144
it sounds like the wording in the documentation should be clarified because a user installing unpinned dependencies will run into a problem with old notebooks.
Suggested improvement
Instead of:
If you only need to use the classic notebook, you can use
it could say
If you only need to use the classic notebook (i.e., notebook<7), you can use
(I'd also suggest a better error message somewhere)
That seems reasonable, though I find that whole section somewhat convoluted. Probably should be fixed in figure intro as well. Any interest in making a PR?
An important detail to us that we've noticed is that with notebook==7 the fix:
%matplotlib widget
isn't sufficient for us. We're interested in more than just basic interactivity (e.g., panning, zooming) but also live updating, for example, the plot changing as data streams in. This is the closest issue to our problem that we've found: matplotlib/ipympl#290
Probably this isn't relevant to the PR in progress; I'm just making a note of functionality that has changed between versions 6 and 7 of notebook.
@qci-amos ipympl should support all of that functionality. You may need to create the figure a cell before the cell where you stream data into it though.
Activity
jklymak commentedon Oct 19, 2023
That seems reasonable, though I find that whole section somewhat convoluted. Probably should be fixed in figure intro as well. Any interest in making a PR?
qci-amos commentedon Oct 19, 2023
I'm not qualified to make a PR! 😄 I'm just wearing a devops hat here.
mattymo30 commentedon Oct 19, 2023
I would like to take a stab at this, at the very least fixing the wording suggested above.
qci-amos commentedon Oct 23, 2023
An important detail to us that we've noticed is that with
notebook==7the fix:isn't sufficient for us. We're interested in more than just basic interactivity (e.g., panning, zooming) but also live updating, for example, the plot changing as data streams in. This is the closest issue to our problem that we've found: matplotlib/ipympl#290
Probably this isn't relevant to the PR in progress; I'm just making a note of functionality that has changed between versions 6 and 7 of
notebook.tacaswell commentedon Oct 23, 2023
@qci-amos ipympl should support all of that functionality. You may need to create the figure a cell before the cell where you stream data into it though.
ksunden commentedon Oct 31, 2023
Closed by #27165