make_dvcyaml: skip metrics plots when plot=False (#652)

* make_dvcyaml: skip metrics plots when plot=False

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Delete _dvclive_version.py

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: David de la Iglesia Castro <daviddelaiglesiacastro@gmail.com>

src/dvclive/dvc.py

@@ -91,9 +91,10 @@ def make_dvcyaml(live):
         dvcyaml["metrics"] = [os.path.relpath(live.metrics_file, live.dir)]
     plots = []
     plots_path = Path(live.plots_dir)
-    if live._metrics:
-        metrics_path = (plots_path / Metric.subfolder).relative_to(live.dir).as_posix()
-        metrics_config = {metrics_path: {"x": "step"}}
+    metrics_path = plots_path / Metric.subfolder
+    if metrics_path.exists():
+        metrics_relpath = metrics_path.relative_to(live.dir).as_posix()
+        metrics_config = {metrics_relpath: {"x": "step"}}
         plots.append(metrics_config)
     if live._images:
         images_path = (plots_path / Image.subfolder).relative_to(live.dir)

tests/test_dvc.py

@@ -57,6 +57,16 @@ def test_make_dvcyaml_metrics(tmp_dir):
     }
 
 
+def test_make_dvcyaml_metrics_no_plots(tmp_dir):
+    live = Live()
+    live.log_metric("bar", 2, plot=False)
+    make_dvcyaml(live)
+
+    assert load_yaml(live.dvc_file) == {
+        "metrics": ["metrics.json"],
+    }
+
+
 def test_make_dvcyaml_summary(tmp_dir):
     live = Live()
     live.summary["bar"] = 2