sagemath/sage

Don't add each piece of piecewise functions to the legend

orlitzky opened this issue · 7 comments

orlitzky commented

This was separated from #11225. The fix is given in AskSage 925 by Douglas McNeil (dsm).

CC: @kcrisman

Component: graphics

Author: Douglas McNeil, Michael Orlitzky

Reviewer: Karl-Dieter Crisman

Merged: sage-5.0.beta9

Issue created by migration from https://trac.sagemath.org/ticket/12651

orlitzky commented

Author: Douglas McNeil, Michael Orlitzky

orlitzky commented

Fix the duplicate legend entries and add a doctest

orlitzky commented
comment:3

Attachment: sage-trac_12651.patch.gz

First patch had two lines with trailing whitespace, I just removed them.

kcrisman commented
comment:4

It seems like everything is okay here. Certainly it fixes the problem, other keywords do apply to all pieces of the function.

When did the :trac: thing come in? This sounds vaguely familiar... at any rate, it works very nicely in the live doc!

kcrisman commented

Reviewer: Karl-Dieter Crisman

orlitzky commented
comment:5

Replying to @kcrisman:

When did the :trac: thing come in? This sounds vaguely familiar... at any rate, it works very nicely in the live doc!

5.0.beta4, apparently. Psyched that I no longer have to worry about word-wrapping "trac #12345" in the doctests.

jdemeyer commented

Merged: sage-5.0.beta9