Ticket #5594 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

[with patch, positive review] better error message for list_plot

Reported by: jason Owned by: was
Priority: minor Milestone: sage-4.1
Component: graphics Keywords:
Cc: jhpalmieri Work issues:
Report Upstream: Reviewers: Nick Alexander
Authors: John Palmieri Merged in: sage-4.1.alpha0
Dependencies: Stopgaps:

Description

We might consider testing whether the second argument to list_plot (which is "plotjoined", should be boolean) is a list or tuple, and then print a warning, because perhaps someone ran "list_plot([list1], [list2])" without meaning to.

Attachments

trac_5594.patch Download (1.5 KB) - added by jhpalmieri 4 years ago.

Change History

comment:1 Changed 4 years ago by jhpalmieri

  • Summary changed from better error message for list_plot to [with patch, needs review] better error message for list_plot

This patch raises a TypeError if plotjoined is a list or a tuple.

Changed 4 years ago by jhpalmieri

comment:2 Changed 4 years ago by ncalexan

  • Reviewers set to Nick Alexander
  • Summary changed from [with patch, needs review] better error message for list_plot to [with patch, positive review] better error message for list_plot
  • Authors set to John Palmieri

Fine by me.

comment:3 Changed 4 years ago by rlm

  • Status changed from new to closed
  • Resolution set to fixed
  • Merged in set to sage-4.1.alpha0
Note: See TracTickets for help on using tickets.