From fc0c553e11329d41110a3ee4a068a62fa844e4aa Mon Sep 17 00:00:00 2001 From: Linlang Date: Sun, 20 Nov 2022 00:15:57 +0800 Subject: [PATCH 1/2] fix_qrun_error --- qlib/workflow/recorder.py | 1 + setup.py | 2 ++ 2 files changed, 3 insertions(+) diff --git a/qlib/workflow/recorder.py b/qlib/workflow/recorder.py index 9d82bf0a47..bb35e10542 100644 --- a/qlib/workflow/recorder.py +++ b/qlib/workflow/recorder.py @@ -21,6 +21,7 @@ from mlflow.store.artifact.azure_blob_artifact_repo import AzureBlobArtifactRepository logger = get_module_logger("workflow", logging.INFO) +mlflow.utils.validation.MAX_PARAM_VAL_LENGTH = 1000 class Recorder: diff --git a/setup.py b/setup.py index faf058d631..9ff13d4a51 100644 --- a/setup.py +++ b/setup.py @@ -62,6 +62,8 @@ def get_version(rel_path: str) -> str: "matplotlib>=3.3", "tables>=3.6.1", "pyyaml>=5.3.1", + # To ensure stable operation of the experiment manager, we have limited the version of mlflow, + # and we need to verify whether version 2.0 of mlflow can serve qlib properly. "mlflow>=1.12.1, <=1.30.0", "tqdm", "loguru", From 1b67eb28f083b2ceeb4d5c5ae713e5e981680b48 Mon Sep 17 00:00:00 2001 From: Linlang Date: Sun, 20 Nov 2022 13:37:49 +0800 Subject: [PATCH 2/2] add_description --- qlib/workflow/recorder.py | 1 + 1 file changed, 1 insertion(+) diff --git a/qlib/workflow/recorder.py b/qlib/workflow/recorder.py index bb35e10542..226f0476d7 100644 --- a/qlib/workflow/recorder.py +++ b/qlib/workflow/recorder.py @@ -21,6 +21,7 @@ from mlflow.store.artifact.azure_blob_artifact_repo import AzureBlobArtifactRepository logger = get_module_logger("workflow", logging.INFO) +# mlflow limits the length of log_param to 500, but this caused errors when using qrun, so we extended the mlflow limit. mlflow.utils.validation.MAX_PARAM_VAL_LENGTH = 1000