Skip to content

Cleanup docstrings related to interactive mode#19863

Merged
tacaswell merged 1 commit intomatplotlib:masterfrom
timhoffm:interactive
Apr 6, 2021
Merged

Cleanup docstrings related to interactive mode#19863
tacaswell merged 1 commit intomatplotlib:masterfrom
timhoffm:interactive

Conversation

@timhoffm
Copy link
Copy Markdown
Member

@timhoffm timhoffm commented Apr 4, 2021

No description provided.

@timhoffm timhoffm added this to the v3.4.2 milestone Apr 4, 2021
Comment on lines +1145 to +1146
This function is only intended for use in backends. End users should
use `.pyplot.isinteractive` instead.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

As far as I understand this, interactive mode does not make sense without pyplot (without a figure manager no updates are done anyway).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I suppose this should indicate why, and the one in pyplot should also link back here for backends.

ret = gca().text(*args, **kwargs)
draw_if_interactive()
return ret
return gca().text(*args, **kwargs)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This was outdated. There is no explicit draw_if_interactive() involved anymore.

@tacaswell tacaswell merged commit 007561d into matplotlib:master Apr 6, 2021
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Apr 6, 2021
@timhoffm timhoffm deleted the interactive branch April 6, 2021 05:42
QuLogic added a commit that referenced this pull request Apr 7, 2021
…863-on-v3.4.x

Backport PR #19863 on branch v3.4.x (Cleanup docstrings related to interactive mode)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants