machinelearning icon indicating copy to clipboard operation
machinelearning copied to clipboard

Published 20 hours ago • verified publisher icon dotnet

Bringing Fairlearn - GridSearch to ML.NET

Open jordi1215 opened this issue 3 years ago • 4 comments

Related to: #1912
Related to: Issue 1091 on the Fairlearn Repo

As part of my summer internship project, I have created several classes that will provide a foundation for the further inclusion of Fairlearn algorithms into ML.NET.

For the first PR, I have included an end-to-end working solution for the gridSearch method in Fairlearn which leverages AutoML for parameter searching. The example is included in the Unit test.

We are excited to review your PR. So we can do the best job, please check:

  • [x] There's a descriptive title that will make sense to other developers some time from now.
  • [x] There's associated issues. All PR's should have issue(s) associated - unless a trivial self-evident change such as fixing a typo. You can use the format Fixes #nnnn in your description to cause GitHub to automatically close the issue(s) when your PR is merged.
  • [x] Your change description explains what the change does, why you chose your approach, and anything else that reviewers should know.
  • [x] You have included any necessary tests in the same PR.

jordi1215 avatar Aug 08 '22 18:08 jordi1215

CLA assistant check
All CLA requirements met.

dnfadmin avatar Aug 08 '22 18:08 dnfadmin

This pull request has been automatically marked no-recent-activity because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 more days. Any new comment (by anyone, not necessarily the author) will remove no-recent-activity.

ghost avatar Aug 24 '22 03:08 ghost

@jordi1215 could you please address the feedback and resolve the conflict in this PR? Thanks!

tarekgh avatar Aug 26 '22 16:08 tarekgh

This pull request has been automatically marked no-recent-activity because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 more days. Any new comment (by anyone, not necessarily the author) will remove no-recent-activity.

ghost avatar Sep 09 '22 21:09 ghost

This pull request will now be closed since it had been marked no-recent-activity but received no further activity in the past 14 days. It is still possible to reopen or comment on the pull request, but please note that it will be locked if it remains inactive for another 30 days.

ghost avatar Sep 24 '22 00:09 ghost