Skip to content
Snippets Groups Projects

Correctly set mlflow logging

Merged Yoann Schneider requested to merge correctly-disable-mlflow into main
1 file
+ 5
5
Compare changes
  • Side-by-side
  • Inline
+ 5
5
@@ -258,18 +258,18 @@ def serialize_config(config):
return serialized_config
def start_training(config) -> None:
def start_training(config, mlflow_logging: bool) -> None:
if (
config["training_params"]["use_ddp"]
and not config["training_params"]["force_cpu"]
):
mp.spawn(
train_and_test,
args=(config, True),
args=(config, mlflow_logging),
nprocs=config["training_params"]["nb_gpu"],
)
else:
train_and_test(0, config, True)
train_and_test(0, config, mlflow_logging)
def run():
@@ -286,7 +286,7 @@ def run():
raise MLflowNotInstalled()
if "mlflow" not in config:
start_training(config)
start_training(config, mlflow_logging=False)
else:
labels_path = (
Path(config["dataset_params"]["datasets"][dataset_name]) / "labels.json"
@@ -314,4 +314,4 @@ def run():
dictionary=artifact,
artifact_file=filename,
)
start_training(config)
start_training(config, mlflow_logging=True)
Loading