From 7c52f6573653d4bc227ec64927dde56b28c4185f Mon Sep 17 00:00:00 2001
From: pawel <pawelredzynski@gmail.com>
Date: Sat, 20 Jul 2019 08:54:37 +0200
Subject: [PATCH 1/2] metric: show: xpath should override xpath set after run

---
 dvc/repo/metrics/show.py   |  2 +-
 tests/func/test_metrics.py | 13 +++++++++++++
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/dvc/repo/metrics/show.py b/dvc/repo/metrics/show.py
index 758ba3a47..f82beb5b3 100644
--- a/dvc/repo/metrics/show.py
+++ b/dvc/repo/metrics/show.py
@@ -190,7 +190,7 @@ def _collect_metrics(repo, path, recursive, typ, xpath, branch):
 
         if not typ and isinstance(o.metric, dict):
             t = o.metric.get(o.PARAM_METRIC_TYPE, typ)
-            x = o.metric.get(o.PARAM_METRIC_XPATH, xpath)
+            x = xpath or o.metric.get(o.PARAM_METRIC_XPATH)
         else:
             t = typ
             x = xpath
diff --git a/tests/func/test_metrics.py b/tests/func/test_metrics.py
index 3e2286c88..4159c6461 100644
--- a/tests/func/test_metrics.py
+++ b/tests/func/test_metrics.py
@@ -831,3 +831,16 @@ class TestShouldDisplayMetricsEvenIfMetricIsMissing(object):
             in caplog.text
         )
         assert 0 == ret
+
+
+def test_show_xpath_should_override_stage_xpath(dvc_repo):
+    metric_file = "metric"
+    metric = {"m1": 0.1, "m2": 0.2}
+    with open(metric_file, "w") as fobj:
+        json.dump(metric, fobj)
+
+    dvc_repo.run(cmd="", overwrite=True, metrics=[metric_file])
+    dvc_repo.metrics.modify(metric_file, typ="json", xpath="m2")
+
+    result = dvc_repo.metrics.show(xpath="m1")
+    assert result == {"": {metric_file: [0.1]}}

From f5e0024137be345c424e63ea06f425d0ea986155 Mon Sep 17 00:00:00 2001
From: pared <pawelredzynski@gmail.com>
Date: Thu, 25 Jul 2019 16:33:49 +0200
Subject: [PATCH 2/2] metrics: show: add comment clarifying metric type and
 xpath usage

---
 dvc/repo/metrics/show.py | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/dvc/repo/metrics/show.py b/dvc/repo/metrics/show.py
index f82beb5b3..4b53f2222 100644
--- a/dvc/repo/metrics/show.py
+++ b/dvc/repo/metrics/show.py
@@ -188,10 +188,16 @@ def _collect_metrics(repo, path, recursive, typ, xpath, branch):
         if not o.metric:
             continue
 
+        # NOTE this case assumes that metric has been modified with
+        # `dvc metric modify -t {} -x {}`, so stage file should contain
+        # info what type metric is and what is its xpath
         if not typ and isinstance(o.metric, dict):
             t = o.metric.get(o.PARAM_METRIC_TYPE, typ)
+            # NOTE user might want to check different xpath, hence xpath first
             x = xpath or o.metric.get(o.PARAM_METRIC_XPATH)
         else:
+            # NOTE if metric has not been modified, user needs to provide
+            # both metric type and xpath
             t = typ
             x = xpath