From 1c01236c5d3deb750368ad123202cecea5bf0b2f Mon Sep 17 00:00:00 2001 From: Miles Olson Date: Fri, 22 Nov 2024 10:41:10 -0800 Subject: [PATCH] Move Potential SQA deprecation message (#3108) Summary: Pull Request resolved: https://github.com/facebook/Ax/pull/3108 Reviewed By: Cesar-Cardoso Differential Revision: D66369657 fbshipit-source-id: fad34541c3a4aaf0dfe77b5c4133a0cfd76919eb --- ax/storage/sqa_store/encoder.py | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/ax/storage/sqa_store/encoder.py b/ax/storage/sqa_store/encoder.py index 89b769c18d2..bf66871b2e4 100644 --- a/ax/storage/sqa_store/encoder.py +++ b/ax/storage/sqa_store/encoder.py @@ -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 @@ -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 )