Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
add 'auto' state for boxplot's patch_artist#28420
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
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
saranti commentedJun 19, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I'll look into it Edit: Always using a patch gives us a problem when someone tries to customize the box using |
The point would be to auto-backconvert (with warning) to the "old" form if that happens, as in#24455/#25247. |
PR summary
In#17959, it was generally agreed that it should be simpler to set the facecolor on a boxplot. I went with@timhoffm'sidea to change the default to
patch_artist='auto'
which would use patches if any of the given parameters requires patches.The 2 boxprops parameters that require a Patch artist are
facecolor
andedgecolor
. If any of those 2 are passed,patch_artist
will auto default to True. Otherwise, the rcparam will be respected.PR checklist