Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Clarify docstring of [un]install_repl_displayhook()#22911
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
lib/matplotlib/pyplot.py Outdated
@@ -153,7 +156,7 @@ def post_execute(): | |||
def uninstall_repl_displayhook(): | |||
""" | |||
UninstalltheMatplotlibdisplay hook. | |||
Disconnect fromthe display hook of the current shell. | |||
.. warning:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
While you are here, you can remove this warning entirely, I think: this system does not actually interact with sys.displayhook ever since#4506.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
done.
dc0ca60
to618ecc9
CompareStill not clear to me: Does this have to be a public function? Whenwould a user call it explicitly?
618ecc9
to83701e9
CompareI think it's worth letting@tacaswell have a look before merging, as he's probably the most knowledgeable on this topic. |
It is fun that we have both |
Still not clear to me: Does this have to be a public function? When
would a user call it explicitly?