#13422: Log scale plots should auto-expand axes range to include at least 2 
ticks
-------------------------------------+-------------------------------------
       Reporter:  ppurka             |        Owner:  jason, was
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.3
      Component:  graphics           |   Resolution:
       Keywords:  log plot           |    Merged in:
        Authors:  Punarbasu          |    Reviewers:  Karl-Dieter Crisman,
  Purkayastha                        |  Michael Orlitzky
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/mjo/ticket/13422                 |  ab527592ca2aa50f487ed71db742a7d25cf733d1
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by mjo):

 * commit:  a7e5686ee6519d9c411e7c0e2b6e5a5e34eecfea =>
     ab527592ca2aa50f487ed71db742a7d25cf733d1
 * branch:  u/ppurka/ticket/13422 => u/mjo/ticket/13422
 * reviewer:  Karl-Dieter Crisman => Karl-Dieter Crisman, Michael Orlitzky


Comment:

 The auto-expand is wonderful, but I don't think we should try to second-
 guess the user if he specifies the ticks manually. It's inconsistent with
 the rest of plotting, and annoying to be honest. It also doesn't in any
 way prevent me from creating a misleading plot.

 I'm posting a patch on top of ppurka's that does two things:

 * Removes the error when ticks are specified manually
 * Factors get_vmin_vmax out into a separate method. It "fits" better where
 it was, but by making it a separate method, we can take advantage of the
 documentation/testing stuff that's already in place. I added a docstring
 and a few examples to show how it works.
 ----
 New commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=014f8528362015c4a43b4832b14cc902c45276cb
 014f852]||{{{auto expand log scale to include at least two major
 ticks}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=ab527592ca2aa50f487ed71db742a7d25cf733d1
 ab52759]||{{{Trac #13422: Reviewer suggestions.}}}||

--
Ticket URL: <http://trac.sagemath.org/ticket/13422#comment:26>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to