You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Make facecolor accept sequences and check that the parameter precedence among color, edgecolor and facecolor is reasonable and comparable with scatter, which can take an explicit color via c (equivalent to color here).
For now, I'd refrain from introducing plural aliase. bar() is originally and primarily a style-all-bars-identical function. Per-bar styling was added later on, and I don't think there's a strong need to support this additional use case with an added plural alias.
The text was updated successfully, but these errors were encountered:
Hi Matplotlib team! My partner and I would like to work on this issue as part of a project for our software engineering class. We're looking forward to contributing by improving the consistency of the bar() function’s color parameters. Any guidance or pointers on starting would be greatly appreciated!
Summary
From #29072.
bar()
supportscolor
: color or list of coloredgecolor
: color or list of colorfacecolor
: colori.e.
facecolor
cannot take a sequenceedgecolors
)hist()
signature to support aliases and plural kwargs #28884Proposed fix
Make
facecolor
accept sequences and check that the parameter precedence amongcolor
,edgecolor
andfacecolor
is reasonable and comparable withscatter
, which can take an explicit color viac
(equivalent tocolor
here).For now, I'd refrain from introducing plural aliase.
bar()
is originally and primarily a style-all-bars-identical function. Per-bar styling was added later on, and I don't think there's a strong need to support this additional use case with an added plural alias.The text was updated successfully, but these errors were encountered: