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

Summary: 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
  • Loading branch information
saitcakmak authored and facebook-github-bot committed Mar 18, 2024
1 parent 564cedb commit f04da45
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 f04da45

Please sign in to comment.