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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Stabilize create_trial
#3196
Stabilize create_trial
#3196
Conversation
@@ -183,7 +183,7 @@ def _create_trial(mo_trial: "multi_objective.trial.FrozenMultiObjectiveTrial") - | |||
warnings.simplefilter("ignore", ExperimentalWarning) | |||
trial = create_trial( | |||
state=mo_trial.state, | |||
values=mo_trial.values, | |||
values=mo_trial.values, # type: ignore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FrozenMultiObjectiveTrial.values
is not Optional[Sequence[float]]
. I simply ignore this mismatch because optuna.multi_objective
is entirely deprecated.
optuna/optuna/multi_objective/trial.py
Line 279 in c4791fc
self.values = tuple(trial.intermediate_values.get(i) for i in range(n_objectives)) |
Codecov Report
@@ Coverage Diff @@
## master #3196 +/- ##
==========================================
- Coverage 91.22% 91.22% -0.01%
==========================================
Files 146 146
Lines 11916 11914 -2
==========================================
- Hits 10870 10868 -2
Misses 1046 1046
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks very good!
Let me merge this PR with my one approval since this is a light PR. |
馃敆 #2956
We now stabilize
create_trial
.