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

Remove posterior_transform from info-theoretic acquisition functions #2590

Closed
wants to merge 2 commits into from

Conversation

hvarfner
Copy link
Contributor

Differential Revision: D64698976

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Oct 21, 2024
…ytorch#2590)

Summary: Pull Request resolved: pytorch#2590

Differential Revision: D64698976
hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Oct 22, 2024
…ytorch#2590)

Summary: Pull Request resolved: pytorch#2590

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Oct 22, 2024
…ytorch#2590)

Summary: Pull Request resolved: pytorch#2590

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Oct 22, 2024
…ytorch#2590)

Summary: Pull Request resolved: pytorch#2590

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

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

Copy link

codecov bot commented Oct 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.98%. Comparing base (24f659c) to head (1c39edf).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2590   +/-   ##
=======================================
  Coverage   99.98%   99.98%           
=======================================
  Files         196      196           
  Lines       17345    17352    +7     
=======================================
+ Hits        17343    17350    +7     
  Misses          2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Oct 22, 2024
…ytorch#2590)

Summary: Pull Request resolved: pytorch#2590

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Oct 22, 2024
…ytorch#2590)

Summary: Pull Request resolved: pytorch#2590

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Oct 22, 2024
…2590)

Summary:
Removes `maximize` from some info-theoretic acquisition functions (those that use `get_optimal_samples`). 



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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Oct 22, 2024
…2590)

Summary:
Removes `maximize` from some info-theoretic acquisition functions (those that use `get_optimal_samples`). 



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

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

Carl Hvarfner added 2 commits October 23, 2024 06:53
…samples (pytorch#2576)

Summary:

Added `posterior_transform` arg to get_optimal_samples to enable posterior sampling-based (xES, TestSet IG) acquisition functions with minimization problems. Intended use in one-shot settings.

Reviewed By: saitcakmak

Differential Revision: D64266499
…2590)

Summary:
Removes `maximize` from some info-theoretic acquisition functions (those that use `get_optimal_samples`). 



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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 563cd95.

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