Avoid performance hit from converting dict to text and back again/outline.py

Author: Gerald Young

synfig-studio/plugins/lottie-exporter/Makefile.am

@@ -4,7 +4,8 @@ PLUGIN_NAME = lottie-exporter
 
 EXTRA_FILES = canvas.py \
 			  misc.py \
-			  settings.py
+			  settings.py \
+			  dumb_store.py
 
 plugindir = ${datadir}/synfig/plugins/$(PLUGIN_NAME)
 plugin_DATA = \

synfig-studio/plugins/lottie-exporter/dumb_store.py

@@ -0,0 +1,27 @@
+# pylint: disable=line-too-long
+"""
+Store a value and retrieve it later
+"""
+
+def init():
+    global store_dict
+    store_dict = {}
+
+    global store_next_key
+    store_next_key = int()
+
+
+def put(value):
+    global store_next_key
+    key = str(store_next_key)
+    store_next_key += 1
+
+    global store_dict
+    store_dict[key] = value
+    return key
+
+def get(key):
+    global store_dict
+    if key not in store_dict:
+        return None
+    return store_dict[key]

synfig-studio/plugins/lottie-exporter/lottie-exporter.py

@@ -16,6 +16,7 @@
 from canvas import gen_canvas
 from layers.driver import gen_layers
 import settings
+import dumb_store
 
 
 def write_to(filename, extension, data):
@@ -130,6 +131,7 @@ def init_logs():
     sys.exit()
 else:
     settings.init()
+    dumb_store.init()
     FILE_NAME = sys.argv[1]
     new_file_name = parse(FILE_NAME)
     gen_html(new_file_name)

synfig-studio/plugins/lottie-exporter/properties/shapePropKeyframe/helper.py

@@ -5,7 +5,7 @@
 """
 
 import sys
-import ast
+import dumb_store
 from lxml import etree
 import settings
 from misc import change_axis, get_frame, Vector, is_animated, radial_to_tangent
@@ -28,7 +28,7 @@ def append_path(element, parent, element_name, typ="real"):
         gen_properties_multi_dimensional_keyframed(element_dict, element, 0)
     # Store in lxml element
     element_lxml = etree.Element(element_name)
-    element_lxml.text = str(element_dict)
+    element_lxml.text = dumb_store.put(element_dict)
     parent.append(element_lxml)
 
 
@@ -181,24 +181,24 @@ def get_tangent_at_frame(t1, t2, split_r, split_a, fr):
     # Setting tangent 1
     for chld in t1:
         if chld.tag == "radius_path":
-            dictionary = ast.literal_eval(chld.text)
+            dictionary = dumb_store.get(chld.text)
             r1 = get_vector_at_frame(dictionary, fr)
         elif chld.tag == "theta_path":
-            dictionary = ast.literal_eval(chld.text)
+            dictionary = dumb_store.get(chld.text)
             a1 = get_vector_at_frame(dictionary, fr)
     x, y = radial_to_tangent(r1, a1)
     tangent1 = Vector(x, y)
 
     # Setting tangent 2
     for chld in t2:
         if chld.tag == "radius_path":
-            dictionary = ast.literal_eval(chld.text)
+            dictionary = dumb_store.get(chld.text)
             r2 = get_vector_at_frame(dictionary, fr)
             if not sp_r:
                 # Use t1's radius
                 r2 = r1
         elif chld.tag == "theta_path":
-            dictionary = ast.literal_eval(chld.text)
+            dictionary = dumb_store.get(chld.text)
             a2 = get_vector_at_frame(dictionary, fr)
             if not sp_a:
                 # Use t1's angle

synfig-studio/plugins/lottie-exporter/properties/shapePropKeyframe/outline.py

@@ -5,8 +5,8 @@
 """
 
 import sys
-import ast
 import math
+import dumb_store
 import settings
 from misc import Vector, Hermite
 from synfig.animation import to_Synfig_axis, get_vector_at_frame, get_bool_at_frame, gen_dummy_waypoint
@@ -203,7 +203,7 @@ def get_outline_grow(fr):
         else:
             for chld in og:
                 if chld.tag == "outline_grow_path":
-                    dictionary = ast.literal_eval(chld.text)
+                    dictionary = dumb_store.get(chld.text)
                     val = to_Synfig_axis(get_vector_at_frame(dictionary, fr), "real")
             ret += val
     ret = math.e ** ret
@@ -229,10 +229,10 @@ def get_outline_param_at_frame(composite, fr):
     """
     for child in composite:
         if child.tag == "point_path":
-            dictionary = ast.literal_eval(child.text)
+            dictionary = dumb_store.get(child.text)
             pos = get_vector_at_frame(dictionary, fr)
         elif child.tag == "width_path":
-            dictionary = ast.literal_eval(child.text)
+            dictionary = dumb_store.get(child.text)
             width = to_Synfig_axis(get_vector_at_frame(dictionary, fr), "real")
         elif child.tag == "t1":
             t1 = child[0]

synfig-studio/plugins/lottie-exporter/properties/shapePropKeyframe/polygon.py

@@ -5,7 +5,7 @@
 """
 
 import sys
-import ast
+import dumb_store
 from misc import Vector
 from synfig.animation import get_vector_at_frame, gen_dummy_waypoint
 from properties.multiDimensionalKeyframed import gen_properties_multi_dimensional_keyframed
@@ -82,7 +82,7 @@ def gen_dynamic_list_polygon(lottie, dynamic_list):
                 # Only two childs, one should be animated, other one is path
                 for child in entry:
                     if child.tag == "pos_path":
-                        dictionary = ast.literal_eval(child.text)
+                        dictionary = dumb_store.get(child.text)
                         pos_cur = get_vector_at_frame(dictionary, fr)
                         pos_next = get_vector_at_frame(dictionary, nx_fr)
 

synfig-studio/plugins/lottie-exporter/properties/shapePropKeyframe/region.py

@@ -5,7 +5,7 @@
 """
 
 import sys
-import ast
+import dumb_store
 from synfig.animation import get_vector_at_frame, gen_dummy_waypoint
 from properties.multiDimensionalKeyframed import gen_properties_multi_dimensional_keyframed
 from properties.shapePropKeyframe.helper import insert_dict_at, update_frame_window, update_child_at_parent, append_path, animate_radial_composite, get_tangent_at_frame, convert_tangent_to_lottie, update_frame_set, next_frame, trunc_decimals
@@ -115,7 +115,7 @@ def gen_bline_region(lottie, bline_point):
                 composite = entry[0]
                 for child in composite:
                     if child.tag == "point_path":
-                        dictionary = ast.literal_eval(child.text)
+                        dictionary = dumb_store.get(child.text)
                         pos_cur = get_vector_at_frame(dictionary, fr)
                         pos_next = get_vector_at_frame(dictionary, nx_fr)
                     elif child.tag == "t1":