Skip to content

Commit

Permalink
Update log message in WithDBSettingsBase._load_experiment_and_generat…
Browse files Browse the repository at this point in the history
…ion_strategy (#2287)

Summary:
Pull Request resolved: #2287

The previous message would make me think that the trials were being loaded after the log as opposed to being loaded as part of the experiment. Updated the message to clear the confusion.

Reviewed By: Balandat

Differential Revision: D55044629

fbshipit-source-id: a6117cddf0f70ffa77645678e7545252db518701
  • Loading branch information
saitcakmak authored and facebook-github-bot committed Mar 19, 2024
1 parent 564cedb commit 54996ed
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions ax/service/utils/with_db_settings_base.py
Original file line number Diff line number Diff line change
Expand Up @@ -257,10 +257,10 @@ def _load_experiment_and_generation_strategy(
)
if not isinstance(experiment, Experiment):
raise ValueError("Service API only supports `Experiment`.")
num_trials = len(experiment.trials)
logger.info(
f"Loaded experiment {experiment_name} in "
f"{_round_floats_for_logging(time.time() - start_time)} seconds, "
f"loading trials in mini-batches of {LOADING_MINI_BATCH_SIZE}."
f"Loaded experiment {experiment_name} & {num_trials} trials in "
f"{_round_floats_for_logging(time.time() - start_time)} seconds."
)

try:
Expand Down

0 comments on commit 54996ed

Please sign in to comment.