Skip to content

Commit

Permalink
Rename optimize_acqf_mixed.py to optimize_mixed.py (#2987)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #2987

X-link: pytorch/botorch#2600

The name `optimize_acqf_mixed` conflicts with the optimizer with the same name, which is exposed in `botorch/optim/__init__.py`. This would lead to `botorch.optim.optimize_acqf_mixed` pointing to two different things.

Reviewed By: Balandat

Differential Revision: D65158159

fbshipit-source-id: 285977cd3ddd3c8fbed7beb4b778d896d5251ae6
  • Loading branch information
saitcakmak authored and facebook-github-bot committed Oct 29, 2024
1 parent acb1cc6 commit 9e1f08f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion ax/models/torch/botorch_modular/acquisition.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@
optimize_acqf_discrete_local_search,
optimize_acqf_mixed,
)
from botorch.optim.optimize_acqf_mixed import optimize_acqf_mixed_alternating
from botorch.optim.optimize_mixed import optimize_acqf_mixed_alternating
from botorch.utils.constraints import get_outcome_constraint_transforms
from pyre_extensions import none_throws
from torch import Tensor
Expand Down
2 changes: 1 addition & 1 deletion ax/models/torch/tests/test_acquisition.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@
optimize_acqf_discrete,
optimize_acqf_mixed,
)
from botorch.optim.optimize_acqf_mixed import optimize_acqf_mixed_alternating
from botorch.optim.optimize_mixed import optimize_acqf_mixed_alternating
from botorch.utils.constraints import get_outcome_constraint_transforms
from botorch.utils.datasets import SupervisedDataset
from botorch.utils.testing import MockPosterior
Expand Down

0 comments on commit 9e1f08f

Please sign in to comment.