Skip to content

Conversation

@y0z
Copy link
Member

@y0z y0z commented Sep 27, 2024

Contributor Agreements

Please read the contributor agreements and if you agree, please click the checkbox below.

  • I agree to the contributor agreements.

Tip

Please follow the Quick TODO list to smoothly merge your PR.

Motivation & Description of the changes

Add MoCmaSampler, which implements the CMA-ES variant for multi-objective optimization.

TODO List towards PR Merge

Please remove this section if this PR is not an addition of a new package.
Otherwise, please check the following TODO list:

  • Copy ./template/ to create your package
  • Replace <COPYRIGHT HOLDER> in LICENSE of your package with your name
  • Fill out README.md in your package
  • Add import statements of your function or class names to be used in __init__.py
  • Apply the formatter based on the tips in README.md
  • Check whether your module works as intended based on the tips in README.md

@y0z y0z added the new-package New packages label Sep 27, 2024
@y0z y0z changed the title Feature/mocma Add MoCmaSampler Sep 27, 2024
@y0z y0z marked this pull request as draft September 27, 2024 09:59
@y0z y0z marked this pull request as ready for review September 27, 2024 10:06
@HideakiImamura
Copy link
Member

@nabenabe0928 Could you review this PR?

Copy link
Contributor

@nabenabe0928 nabenabe0928 left a comment

Choose a reason for hiding this comment

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

Almost LGTM!

Copy link
Contributor

@nabenabe0928 nabenabe0928 left a comment

Choose a reason for hiding this comment

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

LGTM

@nabenabe0928 nabenabe0928 merged commit 0f6aa50 into optuna:main Oct 9, 2024
4 checks passed
@y0z y0z deleted the feature/mocma branch October 9, 2024 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-package New packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants