Opened 6 years ago

Last modified 6 years ago

#13828 new enhancement

Clean up arguments to all plotting functions

Reported by: ppurka Owned by: jason, was
Priority: major Milestone: sage-wishlist
Component: graphics Keywords:
Cc: kcrisman, novoselt Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #8570, #13830, #13834, #13576, #15002, #10032 Stopgaps:

Description (last modified by ppurka)

Different plot commands have different arguments or keywords for achieving the same final effect. This is very confusing.

This ticket aims to go through the plot functions one by one and fix the arguments so that the Sage interface remains consistent and uniform across all the plot commands irrespective of the underlying plotting method used (currently matplotlib).

It is conceivable that the changes could be extensive (there are many plot functions) - then the changes can be broken up into other tickets and added to the "Dependencies" of this one.

Change History (11)

comment:1 Changed 6 years ago by ppurka

  • Description modified (diff)

comment:2 Changed 6 years ago by kcrisman

  • Cc kcrisman added

comment:3 Changed 6 years ago by afleckenstein

Is there an example of this? Also, how should we determine what keywords should be used as the standard ones?

comment:4 Changed 6 years ago by ppurka

Unfortunately, there are quite a few such examples. One such I was fixing in #12798 (I intend to break out the color changes to a separate ticket and make that a dependency of this one), there are discussions in #13830, #8570, #13834, and all of this when we haven't even started to look at all the commands in detail.

(I am quite preoccupied for a couple of weeks, so it will take a while before I can come back to all these tickets, and start looking at them in detail.)

comment:5 Changed 6 years ago by novoselt

  • Cc novoselt added

comment:6 Changed 6 years ago by ppurka

Another one mentioned in sage-support:

> When desperate, I add invisible points and use show(). (Unfortunately 
> invisible points depend on your plot type since alpha=0 is for 2d and 
> opacity=0 is for 3d...HINT to developers to change this...)

comment:7 Changed 6 years ago by ppurka

From here:

Somewhere in this set of related tickets we should also unify the usage of figsize between graphics objects and graphics arrays. Maybe graphics arrays need to be a separate file eventually..

comment:8 Changed 6 years ago by ppurka

  • Dependencies set to #8570, #13830, #13834

Let's add some dependencies.

comment:9 Changed 6 years ago by kcrisman

  • Dependencies changed from #8570, #13830, #13834 to #8570, #13830, #13834, #13576

How about #13576 too? Not that any of them have patches.

comment:10 Changed 6 years ago by ppurka

  • Dependencies changed from #8570, #13830, #13834, #13576 to #8570, #13830, #13834, #13576, #15002

Adding #15002 to the list. It arises out of the examples provided by @kcrisman in #13576.

comment:11 Changed 6 years ago by ppurka

  • Dependencies changed from #8570, #13830, #13834, #13576, #15002 to #8570, #13830, #13834, #13576, #15002, #10032
Note: See TracTickets for help on using tickets.