Correct mapping of run_uuid to param value for compare view (mlflow#20)

lu-ohai · Jun 7, 2018 · 1e84a14 · 1e84a14
1 parent f50e612
commit 1e84a14
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git 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
@@ -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)