Named palettes yield unexpected result

I would assume the following two styles to be equivalent

A	B
{ bands: [ {band: 1, palette: ['#000', '#f00']}, // red {band: 2, palette: ['#000', '#0f0']}, // green {band: 3, palette: ['#000', '#00f']} // blue ] }	{ bands: [ {band: 1, palette: 'red'}, {band: 2, palette: 'green'}, {band: 3, palette: 'blue'} ] }

However, they are not and produce totally different thumbnails:

A	B

Is this expected? Should "red" be interpreted as equivalent to ['#000', '#f00']

[manthey]

This was resolved in #858.

[jeffbaumes]

IMO the palette spec could be designed better. The priority order and all the things a string value could represent seems potentially confusing to users.

def getPaletteColors(value):
    """
    Given a list or a name, return a list of colors in the form of a numpy
    array of RGBA.  If a list, each entry is a color name resolvable by either
    PIL.ImageColor.getcolor, by matplotlib.colors, or a 3 or 4 element list or
    tuple of RGB(A) values on a scale of 0-1.  If this is NOT a list, then, if
    it can be parsed as a color, it is treated as ['#000', <value>].  If that
    cannot be parsed, then it is assumed to be a named palette in palettable
    (such as viridis.Viridis_12) or a named palette in matplotlib (including
    plugins).
    :param value: Either a list, a single color name, or a palette name.  See
        above.
    :returns: a numpy array of RGBA value on the scale of [0-255].
    """

As one idea, I’d suggest new specification modes for the different conditionals so the function does not need to interpret opaque strings in so many ways, something like this:

{
 bands: [
    {band: 1, palette: {channel: 'green'}}, // #000 to #0f0
    {band: 2, palette: {matplotlib: 'jet'}},
    {band: 3, palette: {palettable: 'Earth_7'}},
    {band: 4, palette: ['red', '#112233', 'purple', 'green']}, // interpreted as CSS colors
  ]
}

Single-string palettes like palette: 'green' or palette: '#0f0' would not be allowed (or we allow them for now with existing behavior but deprecate it in the future).

This would still let someone do [‘#000’, ‘green’] and get something perhaps unexpected, but I can’t see a way around that because you probably want to support CSS colors in palette arrays.