-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Sklearn update to 1.6+ #5029
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
Sklearn update to 1.6+ #5029
Conversation
Job PR-5029-832278a is done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing it!
@@ -797,8 +798,8 @@ def _calculate_total_resources( | |||
), f"Specified num_cpus per {self.__class__.__name__} is more than the total: {system_num_cpus}" | |||
if user_specified_lower_level_num_gpus is not None: | |||
assert ( | |||
user_specified_lower_level_num_gpus <= system_num_cpus | |||
), f"Specified num_gpus per {self.__class__.__name__} is more than the total: {system_num_cpus}" | |||
user_specified_lower_level_num_gpus <= system_num_gpus |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good to me!
However, the website shows many warnings that need to be addressed:
Mainline for comparison: https://auto.gluon.ai/dev/tutorials/tabular/tabular-essentials.html
Job PR-5029-502c530 is done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks for the contrib!!
Issue #, if available:
#5028
Description of changes:
Update Sklearn to 1.6+. Test failures related with sklearn update
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.