Fix fantasization with FixedNoiseGP and outcome transforms and use FantasizeMixin #2011
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
facebook-github-bot
added
the
CLA Signed
Contributor
|
This pull request was exported from Phabricator. Differential Revision: D49200325 |
Codecov Report
@@ Coverage Diff @@
## main #2011 +/- ##
===========================================
- Coverage 100.00% 99.99% -0.01%
===========================================
Files 179 179
Lines 15798 15806 +8
===========================================
+ Hits 15798 15805 +7
- Misses 0 1 +1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
sdaulton
added a commit
to sdaulton/botorch
that referenced
this pull request
Sep 14, 2023
…ntasizeMixin (meta-pytorch#2011) Summary: 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: D49200325
sdaulton
force-pushed
the
export-D49200325
branch
from
d548d6b to
33d0523
Compare
Contributor
|
This pull request was exported from Phabricator. Differential Revision: D49200325 |
sdaulton
added a commit
to sdaulton/botorch
that referenced
this pull request
Sep 15, 2023
…ntasizeMixin (meta-pytorch#2011) Summary: 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: D49200325
sdaulton
force-pushed
the
export-D49200325
branch
from
33d0523 to
f7319cb
Compare
Contributor
|
This pull request was exported from Phabricator. Differential Revision: D49200325 |
sdaulton
added a commit
to sdaulton/botorch
that referenced
this pull request
Sep 15, 2023
…ntasizeMixin (meta-pytorch#2011) Summary: 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: D49200325
sdaulton
force-pushed
the
export-D49200325
branch
from
f7319cb to
b1a493d
Compare
Contributor
|
This pull request was exported from Phabricator. Differential Revision: D49200325 |
sdaulton
added a commit
to sdaulton/botorch
that referenced
this pull request
Sep 15, 2023
…ntasizeMixin (meta-pytorch#2011) Summary: 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: D49200325
sdaulton
force-pushed
the
export-D49200325
branch
from
b1a493d to
c3c814b
Compare
Contributor
|
This pull request was exported from Phabricator. Differential Revision: D49200325 |
sdaulton
added a commit
to sdaulton/botorch
that referenced
this pull request
Sep 18, 2023
…ntasizeMixin (meta-pytorch#2011) Summary: 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. Reviewed By: Balandat Differential Revision: D49200325
sdaulton
force-pushed
the
export-D49200325
branch
from
c3c814b to
cde2f7d
Compare
Contributor
|
This pull request was exported from Phabricator. Differential Revision: D49200325 |
sdaulton
added a commit
to sdaulton/botorch
that referenced
this pull request
Sep 18, 2023
…ntasizeMixin (meta-pytorch#2011) Summary: 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. Reviewed By: Balandat Differential Revision: D49200325
sdaulton
force-pushed
the
export-D49200325
branch
from
cde2f7d to
77137d4
Compare
Contributor
|
This pull request was exported from Phabricator. Differential Revision: D49200325 |
…ntasizeMixin (meta-pytorch#2011) Summary: 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. Reviewed By: Balandat Differential Revision: D49200325
sdaulton
force-pushed
the
export-D49200325
branch
from
77137d4 to
169cb69
Compare
Contributor
|
This pull request was exported from Phabricator. Differential Revision: D49200325 |
sdaulton
added a commit
to sdaulton/botorch
that referenced
this pull request
Sep 18, 2023
…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: f2150e08009bfdf0f86b0b9e5908610dbb6709ee
sdaulton
added a commit
to sdaulton/botorch
that referenced
this pull request
Sep 18, 2023
Summary: see title. This is causing a failure in the tutorials on meta-pytorch#2011 Differential Revision: D49382057 fbshipit-source-id: f1c62d17d7485ce7e2a381646fad84885ce0d94b
sdaulton
added a commit
to sdaulton/botorch
that referenced
this pull request
Sep 18, 2023
…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: 998c649302579960e6ee2618dfadb87c52d6309b
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
added a commit
to sdaulton/botorch
that referenced
this pull request
Sep 18, 2023
…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
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: 04ba98192b26117c762b4188fae454dca3f8899f
facebook-github-bot
pushed a commit
that referenced
this pull request
Sep 18, 2023
Contributor
|
This pull request has been merged in cbb9ce4. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
This fixes fantasization with FixedNoiseGP and outcome transforms where transformed
noisewas outcome-transformed again.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.fantasizefor handlingXwith size 0 on the -2 dimension.Differential Revision: D49200325