Skip to content

faircode/cli.py's benchmark docstring omits matplotlib from the benchmark extra's package list #244

Description

@yakew7

faircode/cli.py's module docstring says the benchmark command "additionally requires the optional 'benchmark' extra (pip install faircode[benchmark]: scikit-learn + pyyaml + fairlearn)" - that list is now out of date.

Following #235, pyproject.toml's benchmark extra also includes matplotlib (it's what faircode/figures.py needs to render the strategy-comparison PNGs, and its own docstring already claimed it shipped with that extra - it just wasn't actually declared until #235 added it).

faircode/cli.py's docstring wasn't updated to match. One-line fix: add matplotlib to the parenthetical list.

Metadata

Metadata

Assignees

No one assigned

    Labels

    documentationImprovements or additions to documentationgood first issueGood for newcomers

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions