Skip to content

Conversation

tonyhoo
Copy link
Collaborator

@tonyhoo tonyhoo commented Apr 10, 2025

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.

@tonyhoo tonyhoo added this to the 1.3 Release milestone Apr 10, 2025
Copy link

Job PR-5029-832278a is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-5029/832278a/index.html

@tonyhoo tonyhoo changed the title [Draft] Sklearn update to 1.6+ Sklearn update to 1.6+ Apr 11, 2025
@tonyhoo tonyhoo requested review from Innixma and suzhoum April 11, 2025 15:54
Copy link
Contributor

@suzhoum suzhoum left a 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
Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch.

Copy link
Contributor

Choose a reason for hiding this comment

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

+1!

Copy link
Contributor

@Innixma Innixma left a comment

Choose a reason for hiding this comment

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

Copy link

Job PR-5029-502c530 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-5029/502c530/index.html

@tonyhoo tonyhoo requested a review from Innixma April 14, 2025 06:40
Copy link
Contributor

@Innixma Innixma left a 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!!

@tonyhoo tonyhoo merged commit f64e416 into autogluon:master Apr 14, 2025
27 checks passed
@tonyhoo tonyhoo deleted the sklearn_update branch April 14, 2025 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants