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

Update use cases of deprecated fixed noise models #1920

Closed
wants to merge 2 commits into from

Conversation

saitcakmak
Copy link
Contributor

Summary:
Updates use cases of fixed noise models that were deprecated in pytorch/botorch#2052 & elsewhere.

The only remaining usage is botorch_modular_registry. I updated these to point to the parent models, so that they'll be less likely to cause backwards compatibility issues when we delete the models for good.

Differential Revision: D50431137

saitcakmak and others added 2 commits October 18, 2023 16:52
Summary:
Pull Request resolved: facebook#1915

The `use_update` setting of `GenerationStrategy` was deprecated in facebook#1895. This diff follows up to deprecate `update` from `ModelBridge` & `Model` APIs.

All remaining mentions of update raise a `DeprecationWarning` and point the user to `fit` instead: `GenerationNode.use_update`, `ModelBridge.update` & `TorchModel.update`.

Differential Revision: https://internalfb.com/D50351014

fbshipit-source-id: 60cf93cbdd3a5a56ce7fcf682d417b5dcca60e06
Summary:
Pull Request resolved: facebook#1920

Updates use cases of fixed noise models that were deprecated in  pytorch/botorch#2052 & elsewhere.

The only remaining usage is `botorch_modular_registry`. I updated these to point to the parent models, so that they'll be less likely to cause backwards compatibility issues when we delete the models for good.

Reviewed By: esantorella

Differential Revision: D50431137

fbshipit-source-id: 604090019cfff8bc27ae01341a43d80142ef89c8
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Oct 18, 2023
@saitcakmak
Copy link
Contributor Author

pr is disconnected from diff

@saitcakmak saitcakmak closed this Oct 18, 2023
@saitcakmak saitcakmak deleted the export-D50431137 branch October 18, 2023 23:57
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants