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

Update mlos_core API to support with multi-factor optimization #730

Merged
merged 92 commits into from
May 20, 2024

Conversation

motus
Copy link
Member

@motus motus commented May 1, 2024

  • Pass multi-column DataFrame instead of Sequence to BaseOptimizer.register() and other methods that deal with scores
  • Update mlos_bench MlosCoreOptimizer to support the new mlos_core API
  • Update unit tests to work with the new API
  • Add unit tests for end-to-end multi-target optimization

Merge after #726

motus added 30 commits March 22, 2024 14:55
motus added a commit that referenced this pull request May 13, 2024
@motus
Copy link
Member Author

motus commented May 14, 2024

Multi-target optimization in FLAML and optional weights for optimization targets implemented in the next PR #738

@motus motus changed the title Update mlos_core API to support with multi-factor optimization and contexts Update mlos_core API to support with multi-factor optimization May 15, 2024
Copy link
Contributor

@bpkroth bpkroth left a comment

Choose a reason for hiding this comment

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

Let's back out the change to submitting all non-objective scores as context for now. Thanks!

@motus motus enabled auto-merge (squash) May 20, 2024 21:01
Copy link
Contributor

@bpkroth bpkroth left a comment

Choose a reason for hiding this comment

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

Minor comment request and I think so test failures, but LGTM

@motus motus merged commit e2b819f into microsoft:main May 20, 2024
9 of 12 checks passed
@motus motus deleted the sergiym/opt/mobj_core branch May 24, 2024 18:34
bpkroth added a commit that referenced this pull request Jun 3, 2024
…t multi-target optimization for FLAML. (#738)

Summary of changes:
* Pass optional weights for optimization targets in mlos_core
* Implement weighted average for multi-objective optimization in FLAML
* Add more unit tests for multi-objective optimization on mlos_core side

Merge after ~#730~

---------

Co-authored-by: Brian Kroth <bpkroth@users.noreply.github.com>
bpkroth added a commit to bpkroth/MLOS that referenced this pull request Jun 17, 2024
…t multi-target optimization for FLAML. (microsoft#738)

Summary of changes:
* Pass optional weights for optimization targets in mlos_core
* Implement weighted average for multi-objective optimization in FLAML
* Add more unit tests for multi-objective optimization on mlos_core side

Merge after ~microsoft#730~

---------

Co-authored-by: Brian Kroth <bpkroth@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants