Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[ENH]: Expand hist() signature to support aliases and plural kwargs #28884

Copy link
Copy link
Closed
@timhoffm

Description

@timhoffm
Issue body actions

Problem

#28073 added support for arrays on edgecolor, facecolor, linewidth, linestyle, hatch.

#28873 noted that we do not support the arrays for the aliases ec, fc, ... We also do not support plural versions of these parameters edgecolors, facecolors, ...

This is a reasonable expectation.

Proposed solution

  • Alias support should be added.
  • I believe we should come up with a clearer general strategy on singular/plural versions of the kwargs. Some initial thoughts:
    • Make a survey of which functions already support array properties and via which APIs.
    • the singular versions exist as pass-through. They need to be supported anyway. We could (i) only accept singular values there or (ii) also accept sequences. (ii) seems the more user-friendly approach and is implemented in some APIs already.
    • if we support plurals, we need to internally merge the single and plural variants, same with aliases. We likely should make helper functions for this. Can aliases be reused?
hirohira9119

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      Morty Proxy This is a proxified and sanitized view of the page, visit original site.