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

Skip to content

Inconsistence between CalibrationDisplay and over Display #21027

Closed
@glemaitre

Description

@glemaitre

While working on #20999, I could notice that CalibrationDisplay has a name parameter while other displays have a estimator_name parameter. In #20999, I try to create a base class that manages, in the same manner, this parameter and it would be handy to have the same naming.

@thomasjpfan @lucyleeow Do you remember if there is a reason for choosing name instead of estimator_name in this display?
@adrinjalali If the change makes sense, I would submit a PR and it would be great to include before the release to avoid a deprecation cycle for this brand new display.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions