From 1e84a14323826a6b74ca215f56ed1c14fd1a3539 Mon Sep 17 00:00:00 2001 From: Mani Parkhe Date: Thu, 7 Jun 2018 13:55:10 -0700 Subject: [PATCH] Correct mapping of run_uuid to param value for compare view (#20) --- example/quickstart/test.py | 4 ++-- mlflow/server/js/src/components/CompareRunView.js | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/example/quickstart/test.py b/example/quickstart/test.py index 6c823c73ae937..03509d69139a4 100644 --- a/example/quickstart/test.py +++ b/example/quickstart/test.py @@ -1,12 +1,12 @@ import os -from random import random +from random import random, randint from mlflow import log_metric, log_param, log_artifacts if __name__ == "__main__": print("Running test.py") - log_param("param1", 5) + log_param("param1", randint(0, 100)) log_metric("foo", random()) log_metric("foo", random() + 1) diff --git a/mlflow/server/js/src/components/CompareRunView.js b/mlflow/server/js/src/components/CompareRunView.js index 429cd71d0c38d..f77648de9a794 100644 --- a/mlflow/server/js/src/components/CompareRunView.js +++ b/mlflow/server/js/src/components/CompareRunView.js @@ -101,13 +101,13 @@ class Private { paramKeyValueList.push(curKeyValueObj); }); - const mergedParams = Utils.mergeRuns(runInfos.map((r) => r.uuid), paramKeyValueList); + const mergedParams = Utils.mergeRuns(runInfos.map((r) => r.run_uuid), paramKeyValueList); Object.keys(mergedParams).sort().forEach((paramKey) => { const curRow = []; curRow.push(paramKey); runInfos.forEach((r) => { - const curUuid = r.uuid; + const curUuid = r.run_uuid; curRow.push(mergedParams[paramKey][curUuid]); }); rows.push(curRow)