Skip to content

Commit a415d77

Browse files
esantorellameta-codesync[bot]
authored andcommitted
Remove deprecated argument use_update to GenerationStep (#5187)
Summary: Pull Request resolved: #5187 **Context**: * This was deprecated in 2023. * It was actually hard-deprecated because we had `raise DeprecationWarning` (an exception). * It does not do anything, so it is safe to remove. **This PR**: * Removes places where the argument is passed * Does not remove it from a storage backward-compatibility test Reviewed By: saitcakmak Differential Revision: D101822874 fbshipit-source-id: 8a61698dc675a32f9b09c73debcb8b23735dfcab
1 parent 8c8dc6c commit a415d77

2 files changed

Lines changed: 0 additions & 6 deletions

File tree

ax/generation_strategy/generation_node.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -983,7 +983,6 @@ class GenerationStep:
983983
to `generation_strategy.gen` will fail with a `MaxParallelismReached
984984
Exception`, indicating that more trials need to be completed before
985985
generating and running next trials.
986-
use_update: DEPRECATED.
987986
enforce_num_trials: Whether to enforce that only `num_trials` are generated
988987
from the given step. If False and `num_trials` have been generated, but
989988
`min_trials_observed` have not been completed, `generation_strategy.gen`
@@ -1037,7 +1036,6 @@ def __new__(
10371036
should_deduplicate: bool = False,
10381037
generator_name: str | None = None,
10391038
use_all_trials_in_exp: bool = False,
1040-
use_update: bool = False, # DEPRECATED.
10411039
index: int = -1, # Index of this step, set internally.
10421040
suggested_experiment_status: ExperimentStatus | None = None,
10431041
# Deprecated arguments for backwards compatibility.
@@ -1051,9 +1049,6 @@ def __new__(
10511049
Returns:
10521050
A ``GenerationNode`` instance configured with the provided step parameters.
10531051
"""
1054-
if use_update:
1055-
raise DeprecationWarning("`GenerationStep.use_update` is deprecated.")
1056-
10571052
if num_trials < 1 and num_trials != -1:
10581053
raise UserInputError(
10591054
"`num_trials` must be positive or -1 (indicating unlimited) "

ax/utils/testing/modeling_stubs.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -431,7 +431,6 @@ def get_legacy_list_surrogate_generation_step_as_dict() -> dict[str, Any]:
431431
"min_trials_observed": 0,
432432
"completion_criteria": [],
433433
"max_parallelism": 1,
434-
"use_update": False,
435434
"enforce_num_trials": True,
436435
"model_kwargs": {
437436
"surrogate": {

0 commit comments

Comments
 (0)