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

Change skip condition in _get_observation_pairs from multivariate to group #4065

Closed
wants to merge 1 commit into from

Conversation

knshnb
Copy link
Member

@knshnb knshnb commented Oct 11, 2022

Motivation

Description of the changes

@github-actions github-actions bot added the optuna.samplers Related to the `optuna.samplers` submodule. This is automatically labeled by github-actions. label Oct 11, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #4065 (dadb009) into master (1a520bd) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4065      +/-   ##
==========================================
- Coverage   90.06%   90.06%   -0.01%     
==========================================
  Files         160      160              
  Lines       12591    12589       -2     
==========================================
- Hits        11340    11338       -2     
  Misses       1251     1251              
Impacted Files Coverage Δ
optuna/samplers/_tpe/sampler.py 96.94% <ø> (-0.02%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@knshnb knshnb closed this Oct 17, 2022
@knshnb
Copy link
Member Author

knshnb commented Oct 17, 2022

Let me close this issue because I decided to adopt #4079.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
optuna.samplers Related to the `optuna.samplers` submodule. This is automatically labeled by github-actions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants