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.

Sign up for GitHub

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

Sklearn 1.3 update #201

Merged
merged 12 commits into from Dec 20, 2023
Merged

Sklearn 1.3 update #201

merged 12 commits into from Dec 20, 2023

Conversation

mandjevant
Copy link
Contributor

Adjusting of parameters and values to pursue compatibility with sklearn 1.3 update

@mandjevant mandjevant self-assigned this Oct 3, 2023
@mandjevant
Copy link
Contributor Author

mandjevant commented Oct 3, 2023 •
edited

TODO: Split between automated testing update and other updates.
@mohsenari Thank you for your help on this 👍🏼

@mandjevant
Copy link
Contributor Author

This is the proper fix to #198 and other errors. Where #194 was the temp fix.

@mandjevant
Copy link
Contributor Author

Gonna go ahead and merge this. If any issues arise or changes are required, I can implement them over the holidays.

@mandjevant mandjevant merged commit 4bc2864 into hyperopt:master Dec 20, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bjkomer bjkomer Awaiting requested review from bjkomer

Assignees

@mandjevant mandjevant

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@mandjevant