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

Enabling SampleReducingMCAcquisitionFunctions with constraints in input_constructors #1932

Closed
wants to merge 2 commits into from

Conversation

SebastianAment
Copy link
Contributor

Summary: This commit enables the prinicpled constraint treatment via SampleReducingMCAcquisitionFunctions through input_constructors and get_acquisition_function.

Differential Revision: D47365084

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

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Jul 11, 2023
…input_constructors` (pytorch#1932)

Summary:
Pull Request resolved: pytorch#1932

This commit enables the prinicpled constraint treatment via `SampleReducingMCAcquisitionFunctions` through `input_constructors` and `get_acquisition_function`.

Differential Revision: D47365084

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

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Jul 12, 2023
…input_constructors` (pytorch#1932)

Summary:
Pull Request resolved: pytorch#1932

This commit enables the prinicpled constraint treatment via `SampleReducingMCAcquisitionFunctions` through `input_constructors` and `get_acquisition_function`.

Differential Revision: D47365084

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

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Jul 12, 2023
…input_constructors` (pytorch#1932)

Summary:
Pull Request resolved: pytorch#1932

This commit enables the prinicpled constraint treatment via `SampleReducingMCAcquisitionFunctions` through `input_constructors` and `get_acquisition_function`.

Differential Revision: D47365084

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

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

@codecov
Copy link

codecov bot commented Jul 12, 2023

Codecov Report

Merging #1932 (36dc426) into main (a1b38fc) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 36dc426 differs from pull request most recent head 03f7a30. Consider uploading reports for the commit 03f7a30 to get more accurate results

@@           Coverage Diff           @@
##             main    #1932   +/-   ##
=======================================
  Coverage   99.94%   99.94%           
=======================================
  Files         176      176           
  Lines       15480    15500   +20     
=======================================
+ Hits        15472    15492   +20     
  Misses          8        8           
Impacted Files Coverage Δ
botorch/acquisition/input_constructors.py 100.00% <100.00%> (ø)
botorch/acquisition/monte_carlo.py 100.00% <100.00%> (ø)
botorch/acquisition/utils.py 100.00% <100.00%> (ø)
botorch/utils/objective.py 100.00% <100.00%> (ø)

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

SebastianAment and others added 2 commits July 13, 2023 05:32
Summary:
Pull Request resolved: pytorch#1931

This commit separates out the `compute_best_feasible_f` function from `qNEI` as a utility function in order to use it in the `input_constructors` and `get_acquisition_function` (follow-up).

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

fbshipit-source-id: 4b89d25c02b7ac5d56a219b8d15dd4dd17599acf
…input_constructors` (pytorch#1932)

Summary:
Pull Request resolved: pytorch#1932

This commit enables the prinicpled constraint treatment via `SampleReducingMCAcquisitionFunctions` through `input_constructors` and `get_acquisition_function`.

Reviewed By: esantorella

Differential Revision: D47365084

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

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Jul 13, 2023
…input_constructors` (pytorch#1932)

Summary:
Pull Request resolved: pytorch#1932

This commit enables the prinicpled constraint treatment via `SampleReducingMCAcquisitionFunctions` through `input_constructors` and `get_acquisition_function`.

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

fbshipit-source-id: 0af2a9e3c046262a2c988251c8776407e7a0aa91
SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Jul 13, 2023
…input_constructors` (pytorch#1932)

Summary:
Pull Request resolved: pytorch#1932

This commit enables the prinicpled constraint treatment via `SampleReducingMCAcquisitionFunctions` through `input_constructors` and `get_acquisition_function`.

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

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

This pull request has been merged in ef5a939.

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