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

Fix pipelining bug in fairlearn algorithms #323

Merged
merged 3 commits into from
Jul 14, 2022
Merged

Conversation

hoffmansc
Copy link
Collaborator

@hoffmansc hoffmansc commented Jul 14, 2022

move logic from __init__ to allow clone

Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request fixes 4 alerts when merging b198f86 into 17f0aba - view on LGTM.com

fixed alerts:

  • 4 for Unused import

Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
@hoffmansc hoffmansc requested a review from mnagired July 14, 2022 16:28
@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request fixes 4 alerts when merging 8f6d4c7 into 17f0aba - view on LGTM.com

fixed alerts:

  • 4 for Unused import

Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
@hoffmansc hoffmansc merged commit faa75ee into master Jul 14, 2022
@hoffmansc hoffmansc deleted the fairlearn_init_fix branch July 14, 2022 17:42
@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request fixes 4 alerts when merging 24cb4bf into 17f0aba - view on LGTM.com

fixed alerts:

  • 4 for Unused import

Illia-Kryvoviaz pushed a commit to Illia-Kryvoviaz/AIF360 that referenced this pull request Jun 7, 2023
* move logic from __init__ to allow clone
* add classes_ to GridSearchReduction
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.

2 participants