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

Support observed noise in MixedSingleTaskGP #2054

Closed
wants to merge 3 commits into from

Conversation

saitcakmak
Copy link
Contributor

Summary: Supporting observed noise was blocked on SingleTaskGP not supporting train_Yvar. We can easily support it after #2052

Differential Revision: D50394746

@facebook-github-bot facebook-github-bot added CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported labels Oct 18, 2023
@facebook-github-bot
Copy link
Contributor

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

@codecov
Copy link

codecov bot commented Oct 18, 2023

Codecov Report

Merging #2054 (93597dc) into main (a424f6f) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 93597dc differs from pull request most recent head c29dd4a. Consider uploading reports for the commit c29dd4a to get more accurate results

@@            Coverage Diff            @@
##              main     #2054   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          179       179           
  Lines        15847     15802   -45     
=========================================
- Hits         15847     15802   -45     
Files Coverage Δ
botorch/models/gp_regression.py 100.00% <100.00%> (ø)
botorch/models/gp_regression_fidelity.py 100.00% <100.00%> (ø)
botorch/models/gp_regression_mixed.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

@facebook-github-bot
Copy link
Contributor

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

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

Supporting observed noise was blocked on `SingleTaskGP` not supporting `train_Yvar`. We can easily support it after pytorch#2052

Reviewed By: esantorella

Differential Revision: D50394746

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

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

saitcakmak and others added 3 commits October 18, 2023 22:48
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: 42d2e94ae2114c0311e21194930039e64aec4092
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
Summary:
Pull Request resolved: pytorch#2054

Supporting observed noise was blocked on `SingleTaskGP` not supporting `train_Yvar`. We can easily support it after pytorch#2052

Reviewed By: esantorella

Differential Revision: D50394746

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 818e1c9.

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