Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

rcomer
Copy link
Member

@rcomer rcomer commented Jan 4, 2025

PR summary

Fixes #25274. The final figure using that code example now looks like

image

When add_container is called, the container gains the removal method which then gets called within Container.remove.

Thanks to @Higgs32584 for the debugging notes, which got me there a lot faster.

PR checklist

@rcomer rcomer added PR: bugfix Pull requests that fix identified bugs topic: plotting methods labels Jan 4, 2025
@rcomer rcomer added this to the v3.10.1 milestone Jan 4, 2025
@scottshambaugh scottshambaugh merged commit 5a43452 into matplotlib:main Jan 6, 2025
46 checks passed
@scottshambaugh
Copy link
Contributor

@rcomer does this look like the same issue? Some conversation saying it might be #25572

meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jan 6, 2025
scottshambaugh added a commit that referenced this pull request Jan 6, 2025
…401-on-v3.10.x

Backport PR #29401 on branch v3.10.x (FIX: add errorbars with `add_container`)
@rcomer
Copy link
Member Author

rcomer commented Jan 6, 2025

@scottshambaugh I’m afraid I don’t know the first thing about how radio/check buttons work, though Tom's comment at #25572 (comment) suggests it's somewhat different.

@rcomer rcomer deleted the errorbar-remove branch January 6, 2025 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: bugfix Pull requests that fix identified bugs topic: plotting methods
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: .remove() on ErrorbarContainer object does not remove the corresponding item from the legend
3 participants