Commit 9037c4b
Rewrite optimize() to use ax.api.client.Client, remove OptimizationLoop (#5143)
Summary:
Pull Request resolved: #5143
**Goal:** Eliminate legacy code that keeps other legacy code alive. Even though managed loop is deprecated, it has internal use cases. It seemed easier to migrate it then to try to clean up every single use case.
The `OptimizationLoop` class in `managed_loop.py` was a legacy managed optimization loop that used `choose_generation_strategy_legacy`. This diff:
- Rewrites the deprecated `optimize()` function to use the modern `ax.api.client.Client` API
- Removes the `OptimizationLoop` class entirely (no external callers)
- Splits `managed_loop` into its own BUCK target with minimal dependencies
- Updates tests and LLM rules to reflect the changes
The `optimize()` function signature and return type are preserved for backward compatibility with existing callers.
Reviewed By: esantorella
Differential Revision: D99497269
fbshipit-source-id: dc27a30e0637f62baa05714e4ab4ef120295abea1 parent b0e6ea4 commit 9037c4b
3 files changed
Lines changed: 360 additions & 612 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
8 | | - | |
9 | | - | |
10 | | - | |
11 | | - | |
12 | | - | |
0 commit comments