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

AxClient.get_best_trial(...) does not account for optimization_config parameter #2927

Open
vassilyl opened this issue Oct 19, 2024 · 1 comment
Assignees

Comments

@vassilyl
Copy link

Steps:

  • Complete a multi-objective experiment with Service API.
  • To get best predicted value for a single objective, create a new OptimizationConfig with the objective and pass it to AxClient.get_best_trial.

Expected:

The call returns best parameterization and predictions.

Observed:

An exception is raised from the implementation method '_get_best_trial`.

"Please use `get_pareto_optimal_parameters` for multi-objective "

since AxClient.get_best_trial(...) doesn't pass optimization_config parameter value to implementation

def get_best_trial(

I've noticed similar issue for

def get_pareto_optimal_parameters(

@mgarrard
Copy link
Contributor

Hi @vassilyl could you provide the full repro for where you create a new OptimizationConfig? I'm curious if maybe the optimization config isn't being saved to the experiment.

@mgarrard mgarrard self-assigned this Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants