-
Notifications
You must be signed in to change notification settings - Fork 321
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
ExternalGenerationNode #2266
ExternalGenerationNode #2266
Conversation
This pull request was exported from Phabricator. Differential Revision: D54500745 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2266 +/- ##
==========================================
+ Coverage 94.89% 94.90% +0.01%
==========================================
Files 479 481 +2
Lines 46946 47060 +114
==========================================
+ Hits 44548 44661 +113
- Misses 2398 2399 +1 ☔ View full report in Codecov by Sentry. |
This pull request was exported from Phabricator. Differential Revision: D54500745 |
0b5eebf
to
959fc76
Compare
Summary: Implements a `ExternalGenerationNode` class, which defers to arbitrary non-Ax / BoTorch based methods for candidate generation. This class mostly retains the signature of the `GenerationNode` for compatibility with `GenerationStrategy`, and requires the user to implement only the necessary bits. Since it is compatible with `GenerationStrategy`, it can be easily combined with other `GenerationNode`, e.g., with Sobol to use a shared initialization with other generation strategies. This makes it particularly suitable for benchmarking against other methods from Ax. The user needs to implement `__init__`, `update_model_state` and `get_next_trial_parameters`. Differential Revision: D54500745
Summary: Pull Request resolved: facebook#2266 Implements a `ExternalGenerationNode` class, which defers to arbitrary non-Ax / BoTorch based methods for candidate generation. This class mostly retains the signature of the `GenerationNode` for compatibility with `GenerationStrategy`, and requires the user to implement only the necessary bits. Since it is compatible with `GenerationStrategy`, it can be easily combined with other `GenerationNode`, e.g., with Sobol to use a shared initialization with other generation strategies. This makes it particularly suitable for benchmarking against other methods from Ax. The user needs to implement `__init__`, `update_model_state` and `get_next_trial_parameters`. Reviewed By: Balandat Differential Revision: D54500745
This pull request was exported from Phabricator. Differential Revision: D54500745 |
959fc76
to
e652d92
Compare
This pull request has been merged in 2081f6f. |
Summary:
Implements a
ExternalGenerationNode
class, which defers to arbitrary non-Ax / BoTorch based methods for candidate generation. This class mostly retains the signature of theGenerationNode
for compatibility withGenerationStrategy
, and requires the user to implement only the necessary bits. Since it is compatible withGenerationStrategy
, it can be easily combined with otherGenerationNode
, e.g., with Sobol to use a shared initialization with other generation strategies. This makes it particularly suitable for benchmarking against other methods from Ax.The user needs to implement
__init__
,update_model_state
andget_next_trial_parameters
.Differential Revision: D54500745