FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

v1.1.1 release candidate testing · Issue #791 · matplotlib/matplotlib · GitHub

Repository navigation

v1.1.1 release candidate testing #791

Closed

Description

Open thread for issues related to testing the v1.1.1 release candidate(s)

Tarballs at https://sourceforge.net/projects/matplotlib/files/matplotlib/matplotlib-1.1.1/

Git clone:

git clone git@github.com:matplotlib/matplotlib.git
git checkout v1.1.x

Activity

  1. ivanov commented on Mar 23, 2012

    Member

    I'm still getting a couple of errors, these may have been mentioned in #779, this is on Ubuntu 12.04

    ======================================================================
    FAIL: matplotlib.tests.test_delaunay.test_gauss.test
    ----------------------------------------------------------------------
    Traceback (most recent call last):
      File "/usr/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
        self.test(*self.arg)
      File "/home/sofia/.local/lib/python2.7/site-packages/matplotlib/testing/decorators.py", line 36, in failer
        result = f(*args, **kwargs)
      File "/home/sofia/.local/lib/python2.7/site-packages/matplotlib/testing/decorators.py", line 140, in do_test
        '(RMS %(rms).3f)'%err)
    ImageComparisonFailure: images not close: /home/sofia/code/matplotlib/result_images/test_delaunay/gauss-lin-con.png vs. /home/sofia/code/matplotlib/result_images/test_delaunay/expected-gauss-lin-con.png (RMS 10.677)
    
    ======================================================================
    FAIL: matplotlib.tests.test_delaunay.test_trig.test
    ----------------------------------------------------------------------
    Traceback (most recent call last):
      File "/usr/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
        self.test(*self.arg)
      File "/home/sofia/.local/lib/python2.7/site-packages/matplotlib/testing/decorators.py", line 36, in failer
        result = f(*args, **kwargs)
      File "/home/sofia/.local/lib/python2.7/site-packages/matplotlib/testing/decorators.py", line 140, in do_test
        '(RMS %(rms).3f)'%err)
    ImageComparisonFailure: images not close: /home/sofia/code/matplotlib/result_images/test_delaunay/trig-lin-con.png vs. /home/sofia/code/matplotlib/result_images/test_delaunay/expected-trig-lin-con.png (RMS 19.105)
    
    ----------------------------------------------------------------------
    Ran 1061 tests in 1013.936s
    
    FAILED (KNOWNFAIL=266, failures=2)
    


    (The known fails are due to not having inkscape)

    the diff images can be found here

  2. cgohlke commented on Mar 23, 2012

    Contributor

    Looks like #576 wasn't backported in full. Sliders are still rendered as triangles when valmin == valinit. See also #581 (comment)

  3. mdboom commented on Mar 23, 2012

    Member

    I made a small mistake in the new mono font rendering (used by the tests). See #793.

  4. cgohlke commented on Mar 23, 2012

    Contributor

    On Windows matplotlib.test() still fails with hundreds of IOError: [Errno 24] Too many open files. Any chance 4cb2aaa could be backported? Probably not that easy since the with statement was introduced in Python 2.5.

  5. mdboom commented on Mar 23, 2012

    Member

    @cgohlke: It would be nice to backport that commit, but as you say it won't be very direct as we'd have to replace the with statements. I'll attempt this -- would you be willing to test any PR on Windows/Python2.5?

  6. mdboom commented on Mar 23, 2012

    Member

    @cgohlke: See #795. I'm not sure it will resolve your issue, but I'm curious to know if it does! I think it should only be merged if we plan to have another release candidate, as it's a fairly significant change.

  7. cgohlke commented on Mar 24, 2012

    Contributor

    @mdboom: Thank you. I tested #795 and it does indeed not solve the problem. Good to know. I don't think #795 (as is) can be applied to v1.1.1 because it is not compatible with Python 2.4 and Python 2.5 would require from __future__ import with_statement in a couple of files.

  8. cgohlke commented on Mar 24, 2012

    Contributor

    As a workaround for the Too many open files errors, the max number of allowed open files can be increased with import ctypes;ctypes.cdll.msvcr90._setmaxstdio(2048). All tests pass on win-amd64-py2.7.

  9. jdh2358 commented on Mar 24, 2012

    CollaboratorAuthor

    @cgohlke thanks for the catch on the slider valmin glitch. I finished the backport in #797

  10. jdh2358 commented on Mar 24, 2012

    CollaboratorAuthor

    @cgohlke re the too many open file handles bug, does #798 help?

  11. cgohlke commented on Mar 24, 2012

    Contributor

    #798 does help: matplotlib.test() passes on win-amd64-py2.7 without increasing maxstdio. Thank you.

  12. mdboom commented on Mar 24, 2012

    Member

    I've updated #795 to remove the few remaining with statements. Can you test on Python 2.5? (Though it doesn't actually solve the underlying issue -- I think #798 is better for that -- we perhaps should just leave well enough alone and not merge #795, though it should help with the number of open file handles, just not on the same scale as #798).

  13. jdh2358 commented on Mar 24, 2012

    CollaboratorAuthor

    @mdboom given the thread safetly issue I mentioned on #798, we should continue testing #795 too. I will test it on python 2.4 later today. Anyone who is a thread expert, please look at the discussion on #798 about thread safety using class level versus instance level caching of the FT2Font instances, and pitch in with any suggestions.

  14. cgohlke commented on Mar 25, 2012

    Contributor

    @mdboom: Where do I find the updated #795? Also, not critical, the PIL images in testing.compare.py are not getting closed, e.g. https://github.com/matplotlib/matplotlib/blob/v1.1.x/lib/matplotlib/testing/compare.py#L219. This seems fixed in master too.

  15. breisfeld commented on May 29, 2012

    platform:
    Windows 7, 64 bit

    python:
    Enthought Python Distribution -- www.enthought.com
    Version: 7.1-1 (32-bit)

    matplotlib:
    http://www.lfd.uci.edu/~gohlke/pythonlibs/#matplotlib
    matplotlib-1.1.1rc.win32-py2.7.‌exe

    With or without the command 'import ctypes;ctypes.cdll.msvcr90._setmaxstdio(2048)',
    I get hundreds of errors like the following when running 'matplotlib.test()':

    ======================================================================
    ERROR: matplotlib.tests.test_legend.test_legend_auto2.test
    ----------------------------------------------------------------------
    Traceback (most recent call last):
      File "C:\Python27\lib\site-packages\nose\case.py", line 187, in runTest
        self.test(*self.arg)
      File "C:\Python27\lib\site-packages\matplotlib\testing\decorators.py", line 36, in failer
        result = f(*args, **kwargs)
      File "C:\Python27\lib\site-packages\matplotlib\testing\decorators.py", line 130, in do_test
        err = compare_images(expected_fname, actual_fname, self._tol, in_decorator=True)
      File "C:\Python27\lib\site-packages\matplotlib\testing\compare.py", line 219, in compare_images
        expectedImage = Image.open( expected ).convert("RGB")
      File "C:\Python27\lib\site-packages\PIL\Image.py", line 1952, in open
        fp = __builtin__.open(fp, "rb")
    IOError: [Errno 2] No such file or directory: 'C:\\Temp\\result_images\\test_legend\\expected-legend_auto2.png'
    
    ----------------------------------------------------------------------
    Ran 1068 tests in 337.868s
    
    FAILED (KNOWNFAIL=534, errors=508)
    
  16. mdboom commented on May 29, 2012

    Member

    @breisfeld: The Windows installer does not include the test data. To get the test data, you'll need to checkout from git.

    If you don't have git and don't want to go through the trouble of getting it installed, you can also get a zip file of the current 1.1.x branch here:

    https://github.com/matplotlib/matplotlib/zipball/v1.1.x

    Extract that somewhere and copy the lib/matplotlib/tests/baseline_images folder to C:\Python27\lib\site-packages\matplotlib\tests.

    The tests should then have their data. (Not tested, as I don't have my Windows VM handy here).

  17. cgohlke commented on May 29, 2012

    Contributor

    @breisfeld: The official release candidate binaries at https://sourceforge.net/projects/matplotlib/files/matplotlib/matplotlib-1.1.1/ include the test data.

  18. efiring commented on Aug 13, 2012

    Member

    Closing; the release has been made.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Release criticalFor bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions


      Back | FazBrowse Home | New Git URL