Skip to content

Commit

Permalink
Do not transform search space in-place in Modelbridge._get_transforme…
Browse files Browse the repository at this point in the history
…d_gen_args (#2386)

Summary:
Pull Request resolved: #2386

The current setup leads to issues when calling `evaluate_acquisition_function()` in between generation steps, see discussion on #2083

Reviewed By: saitcakmak

Differential Revision: D56400492

fbshipit-source-id: 28a71e287ea46b6c17806561c2362aa034108aba
  • Loading branch information
Balandat authored and facebook-github-bot committed Apr 26, 2024
1 parent 6bd1a58 commit 6c55c0f
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions ax/modelbridge/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -690,6 +690,7 @@ def _get_transformed_gen_args(
# TODO(T34225037): replace deepcopy with native clone() in Ax
pending_observations = deepcopy(pending_observations)
fixed_features = deepcopy(fixed_features)
search_space = search_space.clone()

# Transform
for t in self.transforms.values():
Expand Down

0 comments on commit 6c55c0f

Please sign in to comment.