Added new exceptions for swarmplot

Author: sunroofgod

dabest/plot_tools.py

@@ -953,7 +953,7 @@ def _check_errors(
         if not isinstance(self.__jitter, (int, float)):
             raise ValueError("`jitter` must be a scalar or float.")
         if not isinstance(self.__palette, (str, Iterable)):
-            raise ValueError("`palette` must be either a string or an Iterable.")
+            raise ValueError("`palette` must be either a string indicating a color name or an Iterable.")
         if self.__hue is not None and not isinstance(self.__hue, str):
             raise ValueError("`hue` must be either a string or None.")
         if self.__order is not None and not isinstance(self.__order, Iterable):
@@ -978,15 +978,23 @@ def _check_errors(
                         group_i, self.__x
                     )
                     raise IndexError(err)
+
+        if isinstance(self.__palette, str) and self.__palette.strip() == "":
+            err = "`palette` cannot be an empty string. It must be either a string indicating a color name or an Iterable."
+            raise ValueError(err)
         if isinstance(self.__palette, dict):
-            for group_i in self.__palette.keys():
+            # TODO: to add detection of when dict length is less than size of unique_items
+            for group_i, color_i in self.__palette.items():
                 if group_i not in pd.unique(data[color_col]):
                     err = (
                         "{0} in `palette` is not in the '{1}' column of `data`.".format(
                             group_i, color_col
                         )
                     )
                     raise IndexError(err)
+                if isinstance(color_i, str) and color_i.strip() == "":
+                    err = "The color mapping for {0} in `palette` is an empty string. It must contain a color name.".format(group_i)
+                    raise ValueError(err) 
 
         if side.lower() not in ["center", "right", "left"]:
             raise ValueError(

nbs/API/plot_tools.ipynb

@@ -1016,7 +1016,7 @@
     "        if not isinstance(self.__jitter, (int, float)):\n",
     "            raise ValueError(\"`jitter` must be a scalar or float.\")\n",
     "        if not isinstance(self.__palette, (str, Iterable)):\n",
-    "            raise ValueError(\"`palette` must be either a string or an Iterable.\")\n",
+    "            raise ValueError(\"`palette` must be either a string indicating a color name or an Iterable.\")\n",
     "        if self.__hue is not None and not isinstance(self.__hue, str):\n",
     "            raise ValueError(\"`hue` must be either a string or None.\")\n",
     "        if self.__order is not None and not isinstance(self.__order, Iterable):\n",
@@ -1041,15 +1041,23 @@
     "                        group_i, self.__x\n",
     "                    )\n",
     "                    raise IndexError(err)\n",
+    "\n",
+    "        if isinstance(self.__palette, str) and self.__palette.strip() == \"\":\n",
+    "            err = \"`palette` cannot be an empty string. It must be either a string indicating a color name or an Iterable.\"\n",
+    "            raise ValueError(err)\n",
     "        if isinstance(self.__palette, dict):\n",
-    "            for group_i in self.__palette.keys():\n",
+    "            # TODO: to add detection of when dict length is less than size of unique_items\n",
+    "            for group_i, color_i in self.__palette.items():\n",
     "                if group_i not in pd.unique(data[color_col]):\n",
     "                    err = (\n",
     "                        \"{0} in `palette` is not in the '{1}' column of `data`.\".format(\n",
     "                            group_i, color_col\n",
     "                        )\n",
     "                    )\n",
     "                    raise IndexError(err)\n",
+    "                if isinstance(color_i, str) and color_i.strip() == \"\":\n",
+    "                    err = \"The color mapping for {0} in `palette` is an empty string. It must contain a color name.\".format(group_i)\n",
+    "                    raise ValueError(err) \n",
     "\n",
     "        if side.lower() not in [\"center\", \"right\", \"left\"]:\n",
     "            raise ValueError(\n",