Ticket #3392 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

[with patch, positive review] upgrade matplotlib to 0.98.3 release

Reported by: mabshoff Owned by: mabshoff
Priority: major Milestone: sage-3.1.2
Component: packages Keywords:
Cc: Author(s):
Report Upstream: Reviewer(s):
Merged in: Work issues:

Description

matplotlib 0.98.0 is a major release which requires python2.4 and numpy 1.1. It contains significant improvements and may require some advanced users to update their code; see migration and API_CHANGES. We are supporting a maintenance

Attachments

matplotlib-sage.3.patch Download (11.1 KB) - added by jason 2 years ago.
Apply instead of above
trac_3392_doctest-fix.patch Download (0.9 KB) - added by mabshoff 2 years ago.

Change History

Changed 2 years ago by jason

I think we need to upgrade to numpy 1.1.0 and do this upgrade simultaneously. I think they each require the other.

Changed 2 years ago by jason

(see #3390 for the numpy 1.1.0 upgrade spkg).

Changed 2 years ago by tabbott

matplotlib-sage.patch is (at least part of) what is needed to get sage to work with matplotlib 0.98; (they renamed matplotlib.patches.lines to matplotlib.lines).

Ignore matplotlib-sage.2.patch.

Changed 2 years ago by jason

We might as well upgrade to matplotlib 0.98.3 now.

Changed 2 years ago by jason

Apply instead of above

Changed 2 years ago by jason

  • summary changed from upgrade matplotlib to 0.98.0 release to upgrade matplotlib to 0.98.3 release

the matplotlib-sage.3.patch above includes the other two patches and should be applied instead. the .3.patch file also includes several more fixes for things added in 3.1.1.

Changed 2 years ago by mabshoff

  • summary changed from upgrade matplotlib to 0.98.3 release to [with patch, needs review] upgrade matplotlib to 0.98.3 release

Changed 2 years ago by jason

New spkg up at  http://sage.math.washington.edu/home/jason/matplotlib-0.98.3.spkg

This depends on the numpy 1.1 spkg at #3390.

Changed 2 years ago by mabshoff

  • summary changed from [with patch, needs review] upgrade matplotlib to 0.98.3 release to [with patch, positive review] upgrade matplotlib to 0.98.3 release

Spkg and patch look good to me. I integrated Ondrej's fix from #3792 and did some cleanups to SPKG.txt. The new spkg is at

 http://sage.math.washington.edu/home/mabshoff/release-cycles-3.1.2/alpha1/matplotlib-0.98.3.p0.spkg

Cheers,

Michael

Changed 2 years ago by mabshoff

We have some deprecation warning that causes a number of doctests:

        sage -t -long devel/sage/sage/schemes/elliptic_curves/lseries_ell.py # 1 doctests failed
        sage -t -long devel/sage/sage/schemes/elliptic_curves/ell_point.py # 1 doctests failed
        sage -t -long devel/sage/sage/rings/padics/padic_base_generic.py # 1 doctests failed
        sage -t -long devel/sage/sage/rings/polynomial/polynomial_element.pyx # 1 doctests failed
        sage -t -long devel/sage/sage/modules/free_module_element.pyx # 1 doctests failed
        sage -t -long devel/sage/sage/gsl/interpolation.pyx # 1 doctests failed
        sage -t -long devel/sage/sage/gsl/fft.pyx # 1 doctests failed
        sage -t -long devel/sage/sage/gsl/dwt.pyx # 1 doctests failed
        sage -t -long devel/sage/sage/gsl/ode.pyx # 1 doctests failed
        sage -t -long devel/sage/sage/plot/plot.py # 1 doctests failed
        sage -t -long devel/sage/sage/finance/time_series.pyx # 1 doctests failed
        sage -t -long devel/sage/sage/calculus/desolvers.py # 1 doctests failed

Specifically:

sage -t -long devel/sage/sage/finance/time_series.pyx       
**********************************************************************
File "/scratch/mabshoff/release-cycle/sage-3.1.2.alpha1/tmp/time_series.py", line 926:
    sage: v.plot(points=True)
Expected nothing
Got:
    doctest:4821: DeprecationWarning: replace "faceted=False" with "edgecolors='none'"
    <BLANKLINE>
**********************************************************************

Patch coming up.

Cheers,

Michael

Changed 2 years ago by mabshoff

With the following code all doctests pass:

    def _render_on_subplot(self,subplot):
        options = self.options()
        c = to_mpl_color(options['rgbcolor'])
        a = float(options['alpha'])
        s = int(options['pointsize'])
        scatteroptions={}
        if not faceted: scatteroptions['edgecolors'] = 'none'
        subplot.scatter(self.xdata, self.ydata, s=s, c=c, alpha=a, **scatteroptions)

Changed 2 years ago by mabshoff

Changed 2 years ago by mabshoff

  • status changed from new to closed
  • resolution set to fixed

Merged both patches in Sage 3.1.2.alpha1

Changed 2 years ago by jason

uh, as per mabshoff's request that I post something here, that last patch that he merged fixing the doctest looks reasonable, but I haven't actually applied it or doctested it or anything.

Note: See TracTickets for help on using tickets.