Closed a8806e44-c4d4-4baf-918e-1f2679b05e2d closed 14 years ago
Attachment: trac_6875_fill_regression.patch.gz
It still doesn't seem like the first example above works for me---I get a single solid figure. Is it supposed to have a band around it's edge?
Attachment: polar_1.png
The correct output of the first example
Attachment: polar_2.png
The correct output of the second example
Replying to @jasongrout:
It still doesn't seem like the first example above works for me---I get a single solid figure. Is it supposed to have a band around it's edge?
I've attached two images with the correct output of the two examples. Is this what you get with the patch?
I compiled a fresh version of 4.1.1.alpha1 overnight, and everything seems to work on that. So positive review!
(Note to everyone else: the functions above were already in the doctests, but just were not working).
I wish this would have been working last week when I taught finding areas in polar coordinates! I'm glad you found the fix.
Changed author from whuss to Wilfried Huss
Merged trac_6875_fill_regression.patch
.
Merged: Sage 4.1.2.alpha2
Reviewer: Jason Grout
The fill option for polar_plot does not work correctly anymore.
The following two doctests don't produce the supposed output:
This regression has been introduced with the changeset 12287 (Ticket: #5930).
Sage really needs a test framework for plots. Otherwise more and more regressions will creep in, without anybody noticing.
Component: graphics
Keywords: polar_plot, fill
Author: Wilfried Huss
Reviewer: Jason Grout
Merged: Sage 4.1.2.alpha2
Issue created by migration from https://trac.sagemath.org/ticket/6875