Skip to content

Conversation

@jeremiedbb
Copy link
Member

[not really a deprecation clean-up but impossible for me not to preserve title consistency :) ]

non-exact match between feature names at fit and predict/transform should now error.

@jeremiedbb jeremiedbb added No Changelog Needed Quick Review For PRs that are quick to review labels Oct 14, 2022
@jeremiedbb jeremiedbb added this to the 1.2 milestone Oct 14, 2022
Copy link
Member

@lesteve lesteve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ogrisel ogrisel merged commit 04d0e4a into scikit-learn:main Oct 14, 2022
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants