Skip to content

Commit

Permalink
Move Potential SQA deprecation message (#3108)
Browse files Browse the repository at this point in the history
Summary: Pull Request resolved: #3108

Reviewed By: Cesar-Cardoso

Differential Revision: D66369657

fbshipit-source-id: fad34541c3a4aaf0dfe77b5c4133a0cfd76919eb
  • Loading branch information
mpolson64 authored and facebook-github-bot committed Nov 22, 2024
1 parent 1d25234 commit 1c01236
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions ax/storage/sqa_store/encoder.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,11 +84,6 @@ class Encoder:
"""

def __init__(self, config: SQAConfig) -> None:
logger.error(
"ATTENTION: The Ax team is considering deprecating SQLAlchemy storage. "
"If you are currently using SQLAlchemy storage, please reach out to us "
"via GitHub Issues here: https://github.com/facebook/Ax/issues/2975"
)
self.config = config

@classmethod
Expand Down Expand Up @@ -152,6 +147,13 @@ def experiment_to_sqa(self, experiment: Experiment) -> SQAExperiment:
create and store copies of the Trials, Metrics, Parameters,
ParameterConstraints, and Runner owned by this Experiment.
"""

logger.error(
"ATTENTION: The Ax team is considering deprecating SQLAlchemy storage. "
"If you are currently using SQLAlchemy storage, please reach out to us "
"via GitHub Issues here: https://github.com/facebook/Ax/issues/2975"
)

optimization_metrics = self.optimization_config_to_sqa(
experiment.optimization_config
)
Expand Down

0 comments on commit 1c01236

Please sign in to comment.