Skip to content

Conversation

sdaulton
Copy link
Contributor

Summary: see title. This is causing a failure in the tutorials on #2011

Differential Revision: D49382057

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Sep 18, 2023
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

sdaulton added a commit to sdaulton/botorch that referenced this pull request Sep 18, 2023
…#2013)

Summary:
Pull Request resolved: meta-pytorch#2013

see title. This is causing a failure in the tutorials on  meta-pytorch#2011

Reviewed By: Balandat

Differential Revision: D49382057

fbshipit-source-id: 2b88bfeea442e6c1b3d521ee97b4631ff2c77d60
sdaulton and others added 2 commits September 18, 2023 13:09
…ntasizeMixin (meta-pytorch#2011)

Summary:
Pull Request resolved: meta-pytorch#2011

This fixes fantasization with FixedNoiseGP when using outcome transforms----previously, already-transformed noise was transformed again during fantasization.

This also improves the fantasization for batched and batched multi-output models to use the average noise for each batch and output.

This also removes repeated code and uses the logic in `FantasizeMixin.fantasize` for handling `X` with size 0 on the -2 dimension.

This also deprecates the use of `observation_noise` as a boolean argument to fantasize.

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

fbshipit-source-id: 686663284452f02114695d1bb2da973a16c3267e
…#2013)

Summary:
Pull Request resolved: meta-pytorch#2013

see title. This is causing a failure in the tutorials on  meta-pytorch#2011

Reviewed By: Balandat

Differential Revision: D49382057

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

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

@codecov
Copy link

codecov bot commented Sep 18, 2023

Codecov Report

Merging #2013 (0411fa5) into main (fa51038) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 0411fa5 differs from pull request most recent head 15b7861. Consider uploading reports for the commit 15b7861 to get more accurate results

@@            Coverage Diff            @@
##              main     #2013   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          179       179           
  Lines        15798     15807    +9     
=========================================
+ Hits         15798     15807    +9     
Files Changed Coverage Δ
botorch/acquisition/active_learning.py 100.00% <ø> (ø)
botorch/acquisition/knowledge_gradient.py 100.00% <ø> (ø)
botorch/acquisition/max_value_entropy_search.py 100.00% <ø> (ø)
...sition/multi_objective/max_value_entropy_search.py 100.00% <ø> (ø)
botorch/acquisition/multi_step_lookahead.py 100.00% <ø> (ø)
botorch/models/gp_regression.py 100.00% <100.00%> (ø)
botorch/models/gpytorch.py 100.00% <100.00%> (ø)
botorch/models/model.py 100.00% <100.00%> (ø)
botorch/utils/testing.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 has been merged in 9b53b59.

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