Opened 4 years ago

Closed 4 years ago

#36930 closed enhancement (fixed)

Update matplotlib to 1.2 and add Python 3.x ports

Reported by: robitaille@… Owned by: skymoo (Adam Mercer)
Priority: Normal Milestone:
Component: ports Version: 2.1.2
Keywords: Cc: cdeil (Christoph Deil), poorsod@…, eric.lebigot@…, mkr@…, samuelandjw@…, ismo.toijala@…, mndavidoff (Monte Davidoff)
Port: py-matplotlib

Description

Matplotlib 1.2 is now out (http://matplotlib.org/) and supports Python 3.x, so would it be possible to update the port, and add Python 3.1 and 3.2 ports?

Change History (16)

comment:1 Changed 4 years ago by cdeil (Christoph Deil)

  • Cc Deil.Christoph@… added

Cc Me!

comment:2 Changed 4 years ago by skymoo (Adam Mercer)

  • Status changed from new to assigned

I'll and try to get round to this over the weekend.

comment:3 Changed 4 years ago by poorsod@…

  • Cc poorsod@… added

Cc Me!

comment:4 Changed 4 years ago by skymoo (Adam Mercer)

I updated to version 1.2.0 in r99786, at the moment some of the dependencies are missing for python3. I'll work on adding these then get python3 variants added.

comment:5 follow-up: Changed 4 years ago by cdeil (Christoph Deil)

I get pop-up windows during the matplotlib build phase. Is that normal?

comment:6 follow-up: Changed 4 years ago by cdeil (Christoph Deil)

I see two test errors with Python 2.7 on Mac OS X 10.8:

$ nosetests -exe matplotlib
..........E.................................E........................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................./opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/axes.py:1383: DeprecationWarning: use ax.patch instead
  warnings.warn('use ax.patch instead', DeprecationWarning)
......................................................................./opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/gridspec.py:298: UserWarning: This figure includes Axes that are not compatible with tight_layout, so its results might be incorrect.
  warnings.warn("This figure includes Axes that are not "
.................................
======================================================================
ERROR: matplotlib.tests.test_basic.test_simple_knownfail
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/testing/decorators.py", line 47, in failer
    raise KnownFailureTest(msg) # An error here when running nose means that you don't have the matplotlib.testing.noseclasses:KnownFailure plugin in use.
KnownFailureTest: Test known to fail

======================================================================
ERROR: matplotlib.tests.test_dates.test_empty_date_with_year_formatter.test
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/testing/decorators.py", line 72, in test
    self._func()
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/testing/decorators.py", line 47, in failer
    raise KnownFailureTest(msg) # An error here when running nose means that you don't have the matplotlib.testing.noseclasses:KnownFailure plugin in use.
KnownFailureTest: Test known to fail

----------------------------------------------------------------------
Ran 1054 tests in 292.678s

FAILED (errors=2)

comment:7 Changed 4 years ago by eric.lebigot@…

  • Cc eric.lebigot@… added

Cc Me!

comment:8 in reply to: ↑ 5 Changed 4 years ago by skymoo (Adam Mercer)

Replying to Deil.Christoph@…:

I get pop-up windows during the matplotlib build phase. Is that normal?

I don't get any pop-up windows but several python and X threads are spawned which may cause X to launch?

comment:9 in reply to: ↑ 6 Changed 4 years ago by skymoo (Adam Mercer)

Replying to Deil.Christoph@…:

I see two test errors with Python 2.7 on Mac OS X 10.8:

I see a lot of these KnownFailureTest failures as well, I would expect these to fail silently. I'm in the process of clarifying with upstream.

comment:10 Changed 4 years ago by skymoo (Adam Mercer)

I've heard from upstream and it turns out the using the command:

nosetests -exe matplotlib

isn't the correct way of running the tests, as matplotlib has nose extensions that running them using the above doesn't load the extensions hence the errors. If you run:

python -c "import matplotlib; matplotlib.test()"

then I expect the testsuite will pass.

comment:11 Changed 4 years ago by mkr@…

  • Cc mkr@… added

Cc Me!

comment:12 Changed 4 years ago by cdeil (Christoph Deil)

Yes, with python -c "import matplotlib; matplotlib.test()" all tests pass for me also. Wouldn't it be nice if all Python packages had one standard way to test them after installing ... Sorry for the noise.

comment:13 Changed 4 years ago by samuelandjw@…

  • Cc samuelandjw@… added

Cc Me!

comment:14 Changed 4 years ago by ismo.toijala@…

  • Cc ismo.toijala@… added

Cc Me!

comment:15 Changed 4 years ago by mndavidoff (Monte Davidoff)

  • Cc md14-macports@… added

Cc Me!

comment:16 Changed 4 years ago by skymoo (Adam Mercer)

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.