search

sagemath / sage

Main repository of SageMath
https://www.sagemath.org
Other
1.45k stars 481 forks source link

scatter_plot and point2d are essentially the same thing #13830

Open ppurka opened 11 years ago

ppurka commented 11 years ago

Just as the title says - this is a big duplication of code. The class ScatterPlot needs to go and we need to make scatter_plot = point2d.

CC: @jasongrout

Component: graphics

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

kcrisman commented 11 years ago
comment:1

I've been thinking this for a long time, but I didn't know if some people needed different functionality or something... I think the best plan is to make it an alias for point2d but keep the command for those coming from more data-oriented environments, who will be looking first for a command that talks about scatter plots and not think that point would be the "right" thing. That also avoids the whole annoying deprecation process.

I'm cc:ing Jason to see what he thinks (do "owners" get update emails?).

jasongrout commented 11 years ago
comment:2

Somewhere, about a year or two ago, I made the case somewhere to someone (I forget who) that we should just have line() and points(). Scatter plot and list plot should disappear (list plot should degenerate to either line() or points()). I came to this conclusion when I was developing the tutorials for the PREP workshop, and made a conscious decision to not use list_plot or scatter_plot.

So I'm glad we agree :).

jasongrout commented 11 years ago
comment:3

And yes, owners do get emails...

jasongrout commented 11 years ago
comment:4

aha: #8570 comment:7

jasongrout commented 11 years ago
comment:5

(actually, see the entire discussion on #8570)