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

ExternalGenerationNode #2266

Closed
wants to merge 1 commit into from
Closed

Conversation

saitcakmak
Copy link
Contributor

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

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

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

@codecov-commenter
Copy link

codecov-commenter commented Mar 12, 2024

Codecov Report

Attention: Patch coverage is 99.14530% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 94.90%. Comparing base (c695be6) to head (b2fa94f).

❗ Current head b2fa94f differs from pull request most recent head e652d92. Consider uploading reports for the commit e652d92 to get more accurate results

Files Patch % Lines
ax/modelbridge/generation_node.py 83.33% 1 Missing ⚠️
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.
📢 Have feedback on the report? Share it here.

@facebook-github-bot
Copy link
Contributor

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

saitcakmak added a commit to saitcakmak/Ax that referenced this pull request Mar 13, 2024
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
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 2081f6f.

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.

3 participants