Skip to content

Commit c4ee9ff

Browse files
authored
Fixed log_param error (microsoft#1362)
* fix_qrun_error * add_description
1 parent cc01812 commit c4ee9ff

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

qlib/workflow/recorder.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@
2121
from mlflow.store.artifact.azure_blob_artifact_repo import AzureBlobArtifactRepository
2222

2323
logger = get_module_logger("workflow", logging.INFO)
24+
# mlflow limits the length of log_param to 500, but this caused errors when using qrun, so we extended the mlflow limit.
25+
mlflow.utils.validation.MAX_PARAM_VAL_LENGTH = 1000
2426

2527

2628
class Recorder:

setup.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,8 @@ def get_version(rel_path: str) -> str:
6262
"matplotlib>=3.3",
6363
"tables>=3.6.1",
6464
"pyyaml>=5.3.1",
65+
# To ensure stable operation of the experiment manager, we have limited the version of mlflow,
66+
# and we need to verify whether version 2.0 of mlflow can serve qlib properly.
6567
"mlflow>=1.12.1, <=1.30.0",
6668
"tqdm",
6769
"loguru",

0 commit comments

Comments
 (0)