Skip to content

Remove self._renderer from AnnotationBbox and ConnectionPatch#27806

Merged
QuLogic merged 1 commit intomatplotlib:mainfrom
ksunden:rm_renderer
Feb 21, 2024
Merged

Remove self._renderer from AnnotationBbox and ConnectionPatch#27806
QuLogic merged 1 commit intomatplotlib:mainfrom
ksunden:rm_renderer

Conversation

@ksunden
Copy link
Copy Markdown
Member

@ksunden ksunden commented Feb 20, 2024

PR summary

As far as I can tell, these are unused, and cause problems for pickling

Closes #27805

See #27805 (comment) for more details.

PR checklist

As far as I can tell, these are unused, and cause problems for pickling

Closes matplotlib#27805
@ksunden ksunden added the topic: pickle Issues to do with pickling label Feb 20, 2024
Copy link
Copy Markdown
Member

@timhoffm timhoffm left a comment

Choose a reason for hiding this comment

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

I agree, that they seem unused.

Copy link
Copy Markdown
Member

@QuLogic QuLogic left a comment

Choose a reason for hiding this comment

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

i can't find any use for it either.

@QuLogic QuLogic merged commit 631e171 into matplotlib:main Feb 21, 2024
@QuLogic QuLogic added this to the v3.9.0 milestone Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: pickle Issues to do with pickling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Saving a figure with indicate_inset_zoom to pdf and then pickling it causes TypeError

3 participants