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

Rename optimize_acqf_mixed.py to optimize_mixed.py #2987

Closed
wants to merge 1 commit into from

Conversation

saitcakmak
Copy link
Contributor

Summary:
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.

Differential Revision: D65158159

Summary:
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.

Differential Revision: D65158159
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Oct 29, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D65158159

facebook-github-bot pushed a commit to pytorch/botorch that referenced this pull request Oct 29, 2024
Summary:
X-link: facebook/Ax#2987

Pull Request resolved: #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
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 9e1f08f.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants