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.