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

Deprecate FixedNoiseMultiFidelityGP #2053

Closed
wants to merge 2 commits into from

Conversation

saitcakmak
Copy link
Contributor

Summary: Builds on #2052 to deprecate one more model that existed only to support a different likelihood.

Differential Revision: D50393050

@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
@facebook-github-bot
Copy link
Contributor

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

saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Oct 18, 2023
Summary:

Builds on pytorch#2052 to deprecate one more model that existed only to support a different likelihood.

Differential Revision: D50393050
@facebook-github-bot
Copy link
Contributor

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

@codecov
Copy link

codecov bot commented Oct 18, 2023

Codecov Report

Merging #2053 (9b88f7b) into main (a424f6f) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 9b88f7b differs from pull request most recent head 0e4a234. Consider uploading reports for the commit 0e4a234 to get more accurate results

@@            Coverage Diff            @@
##              main     #2053   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          179       179           
  Lines        15847     15806   -41     
=========================================
- Hits         15847     15806   -41     
Files Coverage Δ
botorch/models/gp_regression.py 100.00% <100.00%> (ø)
botorch/models/gp_regression_fidelity.py 100.00% <100.00%> (ø)
botorch/models/gpytorch.py 100.00% <100.00%> (ø)
botorch/models/model.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Oct 18, 2023
Summary:

Builds on pytorch#2052 to deprecate one more model that existed only to support a different likelihood.

Differential Revision: D50393050
saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Oct 18, 2023
Summary:
Pull Request resolved: pytorch#2053

Builds on pytorch#2052 to deprecate one more model that existed only to support a different likelihood.

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

fbshipit-source-id: e5f628d5da8b51dd6aadc1f09e9abd25cfb2ae64
saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Oct 18, 2023
Summary:
Pull Request resolved: pytorch#2053

Builds on pytorch#2052 to deprecate one more model that existed only to support a different likelihood.

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

fbshipit-source-id: 8ee47d06caede6f918bdb1b62f953016e6b5031d
saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Oct 18, 2023
Summary:
Pull Request resolved: pytorch#2053

Builds on pytorch#2052 to deprecate one more model that existed only to support a different likelihood.

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

fbshipit-source-id: dcb19a14bf2ac799720e7c63371766a8c60e6154
@facebook-github-bot
Copy link
Contributor

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

saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Oct 18, 2023
Summary:
Pull Request resolved: pytorch#2053

Builds on pytorch#2052 to deprecate one more model that existed only to support a different likelihood.

Differential Revision: D50393050

fbshipit-source-id: 44dfea109701f5b033889358fdd5222af1c5cda9
saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Oct 18, 2023
Summary:
Pull Request resolved: pytorch#2053

Builds on pytorch#2052 to deprecate one more model that existed only to support a different likelihood.

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

fbshipit-source-id: 710d6c6da7eccb02f944d01996d27fa55c7ed4cb
saitcakmak and others added 2 commits October 18, 2023 22:45
Summary:
Supporting an optional `train_Yvar` argument in `SingleTaskGP` eliminates the need for an additional model class just to have a different likelihood.

The plan for `FixedNoiseGP` is for it to live along with a soft deprecation warning for the foreseeable future.

Differential Revision: D50392099

fbshipit-source-id: 7d11c2e303240316cf8ff7ff5b91c89e0406d2db
Summary:
Pull Request resolved: pytorch#2053

Builds on pytorch#2052 to deprecate one more model that existed only to support a different likelihood.

Reviewed By: Balandat

Differential Revision: D50393050

fbshipit-source-id: 57ef592e2281d2ffdb85a1d3ea06e741c35df904
@facebook-github-bot
Copy link
Contributor

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

saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Oct 19, 2023
Summary:
Pull Request resolved: pytorch#2053

Builds on pytorch#2052 to deprecate one more model that existed only to support a different likelihood.

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

fbshipit-source-id: 8e417e089cdef694f2ec0891b286c18477673037
saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Oct 19, 2023
Summary:
Pull Request resolved: pytorch#2053

Builds on pytorch#2052 to deprecate one more model that existed only to support a different likelihood.

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

fbshipit-source-id: f780903926c108117c7978bdde7c4357a712a50b
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in d98d8fd.

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