Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BUG] fixing constructor non-compliance with sklearn: KNeighborsRegressor #2737

Merged
merged 1 commit into from Jun 8, 2022
Merged

[BUG] fixing constructor non-compliance with sklearn: KNeighborsRegressor #2737

merged 1 commit into from Jun 8, 2022

Conversation

fkiraly
Copy link
Collaborator

@fkiraly fkiraly commented Jun 1, 2022

Part of a fix for #2733, for KNeighborsRegressor.

@fkiraly fkiraly added module:regression regression module: time series regression bugfix Fixes a known bug or removes unintended behavior labels Jun 1, 2022
@fkiraly fkiraly mentioned this pull request Jun 1, 2022
Closed
9 tasks
@fkiraly fkiraly merged commit fbfd0fd into main Jun 8, 2022
@fkiraly fkiraly deleted the KNNReg-kwargs branch June 8, 2022 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aiwalter aiwalter Awaiting requested review from aiwalter

@TonyBagnall TonyBagnall Awaiting requested review from TonyBagnall

Assignees
No one assigned
Labels
bugfix Fixes a known bug or removes unintended behavior module:regression regression module: time series regression
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@fkiraly