Not sure why CI failed the first time, but I re-started it and it seems to be good now.

I this addresses the same issue as #5674 I believe.

Hmm, you're probably right, except that one is a bit more general and adds an additional feature for users, while this one is just straight hard-coding. Probably depends on what the consensus is for that new feature.

I think this can be re-worked to use the new 'CN' color specification method?

Quite likely; I think a bunch of it is already implemented, too. For the stem plots, the baseline was originally red. That's C2 for the classic colour cycle, but C3 for the new one (C2 is green).

Should I go with C2 everywhere, or just special case classic style with C3 everywhere?

I think the red of stem plots should be treated specially, as in a separate
argument. I think when most people are thinking of specifying the color of
their stem plots, they are intending to effect only the stems themselves,
and so that should be what is impacted by the colorcycle and default color.

On Sun, May 1, 2016 at 11:28 PM, Elliott Sales de Andrade <
notifications@github.com> wrote:

Quite likely; I think a bunch of it is already implemented, too. For the
stem plots, the baseline was originally red. That's C2 for the classic
colour cycle, but C3 for the new one (C2 is green).

Should I go with C2 everywhere, or just special case classic style with C3
everywhere?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#5995 (comment)

Yes, there is a separate argument for the stem baseline styling; I'm only referring to the default case if nothing's specified.

Rebased to fix latest v2.x build issues.

This is down to just stem right?

All of my comments on this are non-critical.