Skip to content

Commit 0c35431

Browse files
mpolson64meta-codesync[bot]
authored andcommitted
Fix transitive ObjectiveThreshold import in ax/service/utils/instantiation.py (facebook#5041)
Summary: Pull Request resolved: facebook#5041 Import ObjectiveThreshold from its canonical location (ax.core.outcome_constraint) instead of transitively through ax.core.optimization_config Reviewed By: mgarrard Differential Revision: D96780988 fbshipit-source-id: 2c1f92c1f35119e35c273f7844e9ee1c7d21fcd1
1 parent 038c9cd commit 0c35431

8 files changed

Lines changed: 11 additions & 14 deletions

File tree

ax/analysis/plotly/tests/test_bandit_rollout.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,8 @@
1010
from ax.core.arm import Arm
1111
from ax.core.experiment import Experiment
1212
from ax.core.metric import Metric
13-
from ax.core.optimization_config import Objective, OptimizationConfig
13+
from ax.core.objective import Objective
14+
from ax.core.optimization_config import OptimizationConfig
1415
from ax.core.parameter import ChoiceParameter, ParameterType
1516
from ax.core.search_space import SearchSpace
1617
from ax.utils.common.testutils import TestCase

ax/analysis/tests/test_diagnostics.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,8 @@
2323
from ax.core.data import Data
2424
from ax.core.experiment import Experiment
2525
from ax.core.metric import Metric
26-
from ax.core.optimization_config import Objective, OptimizationConfig
26+
from ax.core.objective import Objective
27+
from ax.core.optimization_config import OptimizationConfig
2728
from ax.core.parameter import ChoiceParameter, ParameterType
2829
from ax.core.search_space import SearchSpace
2930
from ax.generation_strategy.generation_strategy import (

ax/analysis/tests/test_overview.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,8 @@
2424
from ax.core.experiment import Experiment
2525
from ax.core.generator_run import GeneratorRun
2626
from ax.core.metric import Metric
27-
from ax.core.optimization_config import Objective, OptimizationConfig
27+
from ax.core.objective import Objective
28+
from ax.core.optimization_config import OptimizationConfig
2829
from ax.core.outcome_constraint import ScalarizedOutcomeConstraint
2930
from ax.core.parameter import ChoiceParameter, ParameterType
3031
from ax.core.search_space import SearchSpace

ax/analysis/tests/test_results.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,8 @@
1616
from ax.core.experiment import Experiment
1717
from ax.core.generator_run import GeneratorRun
1818
from ax.core.metric import Metric
19-
from ax.core.optimization_config import Objective, OptimizationConfig
19+
from ax.core.objective import Objective
20+
from ax.core.optimization_config import OptimizationConfig
2021
from ax.core.parameter import ChoiceParameter, ParameterType
2122
from ax.core.search_space import SearchSpace
2223
from ax.exceptions.core import UserInputError

ax/benchmark/tests/test_benchmark.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@
7070
get_soo_surrogate,
7171
)
7272
from ax.core.experiment import Experiment
73-
from ax.core.objective import MultiObjective
7473
from ax.core.trial_status import TrialStatus
7574
from ax.early_stopping.strategies.threshold import ThresholdEarlyStoppingStrategy
7675
from ax.generation_strategy.external_generation_node import ExternalGenerationNode

ax/core/optimization_config.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,7 @@
1313

1414
from ax.core.arm import Arm
1515
from ax.core.objective import Objective
16-
from ax.core.outcome_constraint import (
17-
ComparisonOp,
18-
ObjectiveThreshold,
19-
OutcomeConstraint,
20-
)
16+
from ax.core.outcome_constraint import ComparisonOp, OutcomeConstraint
2117
from ax.exceptions.core import UserInputError
2218
from ax.utils.common.base import Base
2319

ax/core/tests/test_experiment.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,10 @@
2525
from ax.core.objective import MultiObjective, Objective
2626
from ax.core.optimization_config import (
2727
MultiObjectiveOptimizationConfig,
28-
ObjectiveThreshold,
2928
OptimizationConfig,
3029
PreferenceOptimizationConfig,
3130
)
32-
from ax.core.outcome_constraint import OutcomeConstraint
31+
from ax.core.outcome_constraint import ObjectiveThreshold, OutcomeConstraint
3332
from ax.core.parameter import (
3433
ChoiceParameter,
3534
DerivedParameter,

ax/service/utils/instantiation.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,9 @@
2222
from ax.core.observation import ObservationFeatures
2323
from ax.core.optimization_config import (
2424
MultiObjectiveOptimizationConfig,
25-
ObjectiveThreshold,
2625
OptimizationConfig,
2726
)
28-
from ax.core.outcome_constraint import OutcomeConstraint
27+
from ax.core.outcome_constraint import ObjectiveThreshold, OutcomeConstraint
2928
from ax.core.parameter import (
3029
ChoiceParameter,
3130
DerivedParameter,

0 commit comments

Comments
 (0)