Set default=True if space/margin passed to register_colors

Author: lukelbd

proplot/colors.py

@@ -613,17 +613,19 @@ def _load_colors(path, ignore_base=True, warn_on_failure=True):
     return loaded
 
 
-def _standardize_colors(input, space='hcl', margin=0.10):
+def _standardize_colors(input, space, margin):
     """
     Standardize the input colors.
 
     Parameters
     ----------
+    input : dict
+        The colors.
     space : optional
         The colorspace used to filter colors.
     margin : optional
-        The proportional margin required for "unique" colors (e.g. 0.10
-        represents 36 hue units, 10 saturation units, 10 luminance units).
+        The proportional margin required for unique colors (e.g. 0.1
+        is 36 hue units, 10 saturation units, 10 luminance units).
     """
     output = {}
     colors = []

proplot/config.py

@@ -642,7 +642,7 @@ def register_cycles(*args, user=None, default=False):
 
 
 @docstring._snippet_manager
-def register_colors(*args, user=None, default=False, space='hcl', margin=0.1, **kwargs):
+def register_colors(*args, user=None, default=False, space=None, margin=None, **kwargs):
     """
     Register named colors. This is called on import.
 
@@ -651,13 +651,15 @@ def register_colors(*args, user=None, default=False, space='hcl', margin=0.1, **
     %(rc.color_args)s
     %(rc.color_params)s
     space : {'hcl', 'hsl', 'hpl'}, optional
-        Ignored if `default` is ``False``. The colorspace used to select "perceptually
-        distinct" colors from the XKCD `color survey <https://xkcd.com/color/rgb/>`_.
+        The colorspace used to select "perceptually distinct" colors from the XKCD
+        `color survey <https://xkcd.com/color/rgb/>`__. Default is ``'hcl'``. If
+        passed then `default` is set to ``True``.
     margin : float, optional
-        Ignored if `default` is ``False``. The margin by which the normalized hue,
-        saturation, and luminance of each XKCD color must differ from the channel
-        values of the other XKCD colors to be deemed "perceptually distinct" and
-        registered. Must be between ``0`` and ``1``. ``0`` will register all colors.
+        The margin by which the normalized hue, saturation, and luminance of each
+        XKCD color must differ from the channel values of the other XKCD colors to
+        be deemed "perceptually distinct" and registered. Must fall between ``0``
+        and ``1`` (``0`` will register all colors). Default is ``0.1``. If
+        passed then `default` is set to ``True``.
     **kwargs
         Additional color name specifications passed as keyword arguments rather
         than positional dictionaries.
@@ -671,6 +673,9 @@ def register_colors(*args, user=None, default=False, space='hcl', margin=0.1, **
     """
     # Add in base colors and CSS4 colors so user has no surprises
     from . import colors as pcolors
+    default = _not_none(space is not None, margin is not None, default)
+    margin = _not_none(margin, 0.1)
+    space = _not_none(space, 'hcl')
     if default:
         pcolors._color_database.clear()  # MutableMapping ensures cache also clears
         for src in (mcolors.CSS4_COLORS, pcolors.COLORS_BASE):
@@ -708,7 +713,7 @@ def register_colors(*args, user=None, default=False, space='hcl', margin=0.1, **
 
         # Add xkcd colors after filtering
         elif cat == 'xkcd':
-            loaded = pcolors._standardize_colors(loaded, space=space, margin=margin)
+            loaded = pcolors._standardize_colors(loaded, space, margin)
             pcolors._color_database.update(loaded)
             pcolors.COLORS_XKCD.update(loaded)