Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#2236 closed defect (fixed)

[with patch, positive review] plot randomizes the endpoints of the interval and causes wiggling in the graph

Reported by: jason Owned by: was
Priority: major Milestone: sage-2.10.2
Component: graphics Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

p=plot(x, (x,-1,1)) p[0][0] == -1 p[0][-1] == 1

They will almost always return false before the patch. After the patch, the two statements should return True always.

Attachments (1)

plot_fix_endpoints.patch​ (1.4 KB) - added by jason 14 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 14 years ago by was

I agree with the suggestion to *not* randomize the endpoints. That's bad.

Changed 14 years ago by jason

comment:2 Changed 14 years ago by jason

  • Summary changed from plot randomizes the endpoints of the interval and causes wiggling in the graph to [with patch, needs review] plot randomizes the endpoints of the interval and causes wiggling in the graph

comment:3 Changed 14 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from new to closed

Merged in Sage 2.10.2.alpha2

comment:4 Changed 14 years ago by jason

For the record, on IRC:

[16:55] <wstein> #2236 -- positive review.

comment:5 Changed 14 years ago by jason

  • Summary changed from [with patch, needs review] plot randomizes the endpoints of the interval and causes wiggling in the graph to [with patch, positive review] plot randomizes the endpoints of the interval and causes wiggling in the graph
Note: See TracTickets for help on using tickets.