Skip to content

Commit

Permalink
Remove MultiFidelityAcquisition (#2520)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #2520

I believe this was originally designed based on the legacy MF models, but it appears to be incompatible with what MBM evolved to be. Here are a few reasons for removing it:
- Some of kwargs that are produced by `MultiFidelityAcquisition.compute_model_dependencies` (e.g., `cost_aware_utility`) are not accepted by the MFKG input constructor.
- The main kwargs that are needed by the MF acquisition functions (`cost_aware_utility`, `project`, `expand`) are readily constructed by the MF input constructors: https://github.com/pytorch/botorch/blob/main/botorch/acquisition/input_constructors.py#L1221
- We strongly discourage subclassing MBM components  and rather encourage expanding the base components to support new use cases. In this case, the MF input constructors seem to have replaced what `MultiFidelityAcquisition` originally aimed to do, while being compatible with the design philosophy.

Discovered while investigating #2514

Reviewed By: Balandat

Differential Revision: D58560934

fbshipit-source-id: fc58675eff4ff81dc0a4a93084e01f8a4c8e0efc
  • Loading branch information
saitcakmak authored and facebook-github-bot committed Jun 14, 2024
1 parent 4ef840b commit b1d3684
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 276 deletions.
100 changes: 0 additions & 100 deletions ax/models/torch/botorch_modular/multi_fidelity.py

This file was deleted.

176 changes: 0 additions & 176 deletions ax/models/torch/tests/test_multi_fidelity.py

This file was deleted.

0 comments on commit b1d3684

Please sign in to comment.