Skip to content

Move Common.set_backend() to TriangleBase.set_backend() #1090

Description

@genedan

Per this comment:

#1082 (comment)

so these tests would be the first time we actually use set_backend on an estimator in the codebase. honestly not sure if it's something we should encourage, or even allow. many estimators have a xp attribute, which could conflict with a manually set backend during subsequent transforming or predicting.

imo we should just take out set_backend from common and put into base (i.e. remove it from estimators). it never made sense to have set_backend on an estimator.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Fields

Priority

Low

Effort

Low

Scope

Codebase

Projects

Status
Backlog

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions