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

Improvement of qBayesianActiveLearningByDisagreement #2457

Closed
wants to merge 1 commit into from

Conversation

hvarfner
Copy link
Contributor

Summary:
Improvement of the implementation of qBayesianActiveLearningByDisagreement

  • Utilizes a Monte Carlo approach for approximating the entropy

  • Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions

  • Can accept posterior transforms

  • get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

Differential Revision: D60308502

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

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

Copy link

codecov bot commented Jul 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.98%. Comparing base (9ddd9eb) to head (5beb446).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2457   +/-   ##
=======================================
  Coverage   99.98%   99.98%           
=======================================
  Files         191      191           
  Lines       16789    16795    +6     
=======================================
+ Hits        16787    16793    +6     
  Misses          2        2           

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

@facebook-github-bot
Copy link
Contributor

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Jul 31, 2024
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Jul 31, 2024
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Jul 31, 2024
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Jul 31, 2024
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Jul 31, 2024
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Aug 1, 2024
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Aug 1, 2024
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Aug 1, 2024
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

Reviewed By: saitcakmak

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Aug 1, 2024
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

Reviewed By: saitcakmak

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

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Aug 1, 2024
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

Reviewed By: saitcakmak

Differential Revision: D60308502
Summary:
Pull Request resolved: pytorch#2457

Improvement of the implementation of qBayesianActiveLearningByDisagreement
- Utilizes a Monte Carlo approach for approximating the entropy
- Does not use concatenate_pending_points, as it is not evident that fantasizing makes sense in the same way as for standard MC acquisition functions
- Can accept posterior transforms

- get_model and get_fully_bayesian_model are used in tests to be similar to other tests (e.g. JES & the subsequent active learning acqfs to enable move to test_helpers

Reviewed By: saitcakmak

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in e44280e.

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