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

fix LearnedObjective base sample shape #2021

Closed
wants to merge 1 commit into from

Conversation

ItsMrLin
Copy link
Contributor

Summary: Explicitly specifcy batch_shape in LearnedObjective's sampler to prevent shared base sample in the same batch.

Reviewed By: Balandat

Differential Revision: D49574252

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

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

@codecov
Copy link

codecov bot commented Sep 24, 2023

Codecov Report

Merging #2021 (4c84ce0) into main (2aef8e2) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 4c84ce0 differs from pull request most recent head 69091e6. Consider uploading reports for the commit 69091e6 to get more accurate results

@@            Coverage Diff            @@
##              main     #2021   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          179       179           
  Lines        15890     15891    +1     
=========================================
+ Hits         15890     15891    +1     
Files Coverage Δ
botorch/acquisition/objective.py 100.00% <100.00%> (ø)

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

ItsMrLin added a commit to ItsMrLin/botorch that referenced this pull request Sep 27, 2023
Summary:

Explicitly specifcy batch_shape in LearnedObjective's sampler to prevent shared base sample in the same batch.

Reviewed By: Balandat, esantorella

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

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

ItsMrLin added a commit to ItsMrLin/botorch that referenced this pull request Sep 27, 2023
Summary:

Explicitly specifcy batch_shape in LearnedObjective's sampler to prevent shared base sample in the same batch.

Reviewed By: Balandat, esantorella

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

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

ItsMrLin added a commit to ItsMrLin/botorch that referenced this pull request Sep 27, 2023
Summary:

Explicitly specifcy batch_shape in LearnedObjective's sampler to prevent shared base sample in the same batch.

Reviewed By: Balandat, esantorella

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

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

ItsMrLin added a commit to ItsMrLin/botorch that referenced this pull request Sep 28, 2023
Summary:

Explicitly specifcy batch_shape in LearnedObjective's sampler to prevent shared base sample in the same batch.

Reviewed By: Balandat, esantorella

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

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

Summary:

Explicitly specifcy batch_shape in LearnedObjective's sampler to prevent shared base sample in the same batch.

Reviewed By: Balandat, esantorella

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in dcb2ba4.

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