Fix vdims in raster - Round two

holoviews/element/raster.py

@@ -1,3 +1,4 @@
+from copy import deepcopy
 from operator import itemgetter
 
 import numpy as np
@@ -610,8 +611,8 @@ def __init__(self, data, kdims=None, vdims=None, **params):
             arrays = [(im.data - r[0]) / (r[1] - r[0]) for r,im in zip(ranges, images)]
             data = np.dstack(arrays)
         if vdims is None:
-            # Same as the class variables, put here to secure the class variable is not used
-            vdims = [Dimension(c, range=(0,1)) for c in "RGB"]
+            # Need to make a deepcopy of the value so the RGB.default is not shared across instances
+            vdims = deepcopy(self.vdims)
         else:
             vdims = list(vdims) if isinstance(vdims, list) else [vdims]
 

holoviews/tests/element/test_raster.py

@@ -3,7 +3,7 @@
 """
 
 import numpy as np
-from holoviews.element import Raster, Image, Curve, QuadMesh, RGB
+from holoviews.element import Raster, Image, Curve, QuadMesh, RGB, HSV
 from holoviews.element.comparison import ComparisonTestCase
 
 class TestRaster(ComparisonTestCase):
@@ -53,6 +53,19 @@ def test_not_using_class_variables_vdims(self):
         cls_vdims = RGB.vdims
         for i, c in zip(init_vdims, cls_vdims):
             assert i is not c
+            assert i == c
+
+class TestHSV(ComparisonTestCase):
+
+    def setUp(self):
+        self.hsv_array = np.random.randint(0, 255, (3, 3, 4))
+
+    def test_not_using_class_variables_vdims(self):
+            init_vdims = HSV(self.hsv_array).vdims
+            cls_vdims = HSV.vdims
+            for i, c in zip(init_vdims, cls_vdims):
+                assert i is not c
+                assert i == c
 
 class TestQuadMesh(ComparisonTestCase):