Changed PlotlyKeyError's superclass to KeyError

But overrode the __str__ method.

Author: nicholas-esterer

packages/python/plotly/_plotly_utils/exceptions.py

@@ -85,12 +85,13 @@ def __init__(self, obj, path, notes=()):
         )
 
 
-class PlotlyKeyError(LookupError):
+class PlotlyKeyError(KeyError):
     """
     KeyErrors are not printed as beautifully as other errors (this is so that
-    {}[''] prints    "KeyError: ''" and not "KeyError:"). So here we subclass
-    LookupError to make a PlotlyKeyError object which will print nicer error
-    messages for KeyErrors.
+    {}[''] prints    "KeyError: ''" and not "KeyError:"). So here we use
+    LookupError's __str__ to make a PlotlyKeyError object which will print nicer
+    error messages for KeyErrors.
     """
 
-    pass
+    def __str__(self):
+        return LookupError.__str__(self)

packages/python/plotly/plotly/basedatatypes.py

@@ -4066,9 +4066,6 @@ def __contains__(self, prop):
         -------
         bool
         """
-        # TODO: We don't want to throw an error in __contains__ because any code that
-        # relies on it returning False will have to be changed (it will have to
-        # have a try except block...).
         prop = BaseFigure._str_to_dict_path(prop)
 
         # Handle remapping