Opened 7 years ago

Closed 6 years ago

#13371 closed enhancement (fixed)

Remove warnings about noninteger bases from sage.plot.plot

Reported by: ppurka Owned by: jason, was
Priority: major Milestone: sage-5.11
Component: graphics Keywords:
Cc: kcrisman Merged in: sage-5.11.rc0
Authors: Punarbasu Purkayastha Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #13693 Stopgaps:

Description (last modified by ppurka)

As per #4529, there was an upstream bug with noninteger bases in logarithmic plots. This has now been fixed upstream and we should remove notes like this from the plot commands and instead encourage the use of noninteger base if the user so desires.

    .. note::

        - Although it is possible to provide a noninteger ``base``, the
          tick labeling and formatting is not ideal. Hence, in case you do
          use noninteger ``base`` for the logarithm, then provide your own
          tick formatter using the option ``tick_formatter``.

This can be done once matplotlib is released with the fix.

Apply to devel/sage: trac_13371-remove_warnings.patch

Attachments (1)

trac_13371-remove_warnings.patch (5.6 KB) - added by ppurka 6 years ago.
apply to devel/sage

Download all attachments as: .zip

Change History (5)

comment:1 Changed 7 years ago by ppurka

  • Dependencies set to #13693

Changed 6 years ago by ppurka

apply to devel/sage

comment:2 Changed 6 years ago by ppurka

  • Authors set to Punarbasu Purkayastha
  • Description modified (diff)
  • Status changed from new to needs_review

Added trivial patch.

comment:3 Changed 6 years ago by tscrim

  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

Looks good to me.

comment:4 Changed 6 years ago by jdemeyer

  • Merged in set to sage-5.11.rc0
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.