Don't initialize uid on traces in frames

This was causing animation bugs

Author: jonmmease

_plotly_utils/basevalidators.py

@@ -1864,11 +1864,18 @@ def validate_coerce(self, v):
 
 class BaseDataValidator(BaseValidator):
 
-    def __init__(self, class_strs_map, plotly_name, parent_name, **kwargs):
+    def __init__(self,
+                 class_strs_map,
+                 plotly_name,
+                 parent_name,
+                 set_uid=False,
+                 **kwargs):
         super(BaseDataValidator, self).__init__(
             plotly_name=plotly_name, parent_name=parent_name, **kwargs)
+
         self.class_strs_map = class_strs_map
         self._class_map = None
+        self.set_uid = set_uid
 
     def description(self):
 
@@ -1957,8 +1964,9 @@ def validate_coerce(self, v):
             v = to_scalar_or_list(res)
 
             # Set new UIDs
-            for trace in v:
-                trace.uid = str(uuid.uuid1())
+            if self.set_uid:
+                for trace in v:
+                    trace.uid = str(uuid.uuid1())
 
         else:
             self.raise_invalid_val(v)

plotly/basedatatypes.py

@@ -119,7 +119,7 @@ class is a subclass of both BaseFigure and widgets.DOMWidget.
         # ### Construct data validator ###
         # This is the validator that handles importing sequences of trace
         # objects
-        self._data_validator = DataValidator()
+        self._data_validator = DataValidator(set_uid=True)
 
         # ### Import traces ###
         data = self._data_validator.validate_coerce(data)