render: avoid deepcopying (iterative#9033)

* render: avoid deepcopying * remove unnecessary line
demo125 · Feb 15, 2023 · be824ed · be824ed
1 parent cbe46cc
commit be824ed
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/dvc/render/convert.py b/dvc/render/convert.py
@@ -29,10 +29,8 @@ def _group_by_rev(datapoints):
 
 
 def to_json(renderer, split: bool = False) -> List[Dict]:
-    from copy import deepcopy
-
     if renderer.TYPE == "vega":
-        grouped = _group_by_rev(deepcopy(renderer.datapoints))
+        grouped = _group_by_rev(renderer.datapoints)
         if split:
             content = renderer.get_filled_template(skip_anchors=["data"])
         else:

diff --git a/dvc/render/converter/vega.py b/dvc/render/converter/vega.py
@@ -1,4 +1,3 @@
-from copy import deepcopy
 from typing import Any, Dict, Iterable, List, Optional, Tuple, Union
 
 from funcy import first, last
@@ -233,7 +232,7 @@ def flat_datapoints(self, revision):  # noqa: C901
         for i, (y_file, y_field) in enumerate(ys):
             if num_xs > 1:
                 x_file, x_field = xs[i]
-            datapoints = deepcopy(file2datapoints.get(y_file, []))
+            datapoints = [d.copy() for d in file2datapoints.get(y_file, [])]
 
             if props_update.get("y", None) == "dvc_inferred_y_value":
                 _update_from_field(