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

Travis Failing in many PRs · Issue #17037 · matplotlib/matplotlib · GitHub

Repository navigation

Travis Failing in many PRs #17037

Closed

Description

Bug report

Travis is showing following errors

======================FAILURES===================================
______________________ test_bbox_inches_tight_raster[pdf] ______________________
[gw1] darwin -- Python 3.7.7 /usr/local/opt/python/bin/python3.7
expected = '/Users/travis/build/matplotlib/matplotlib/result_images/test_bbox_tight/bbox_inches_tight_raster-expected.pdf'
actual = PosixPath('/Users/travis/build/matplotlib/matplotlib/result_images/test_bbox_tight/bbox_inches_tight_raster.pdf')
tol = 0
def _raise_on_image_difference(expected, actual, tol):
tracebackhide = True
err = compare_images(expected, actual, tol, in_decorator=True)
if err:
for key in ["actual", "expected"]:
err[key] = os.path.relpath(err[key])
raise ImageComparisonFailure(
'images not close (RMS %(rms).3f):\n\t%(actual)s\n\t%(expected)s '
% err)
E matplotlib.testing.exceptions.ImageComparisonFailure: images not close (RMS 6.457):
E result_images/test_bbox_tight/bbox_inches_tight_raster_pdf.png
E result_images/test_bbox_tight/bbox_inches_tight_raster-expected_pdf.png

in #17028, #17030 etc.

Bug summary

The error is separate from work done in the prs. Maybe we need to change /bbox_inches_tight_raster-expected_pdf.png ?

Code for reproduction

Check travis build for above prs

Activity

  1. timhoffm commented on Apr 5, 2020

    Member

    This is only failing on OSX. Before changing any reference images, we need to make sure that‘s the correct thing to do. First thing to do is check the generated images to see what‘s different. Somehow we should be able to get those images from Travis, but I don‘t know how.

  2. tacaswell commented on Apr 5, 2020

    Member

    It is also worth looking through the build history (particularly on the master branch) to see if there is a particular commit where it broke. In general the tests are presumed to be correct and the code is presumed to be wrong. Changing the tests generally indicates an API changed an requires a justification.

  3. QuLogic commented on Apr 7, 2020

    Member

    From gitter:
    @tacaswell:

    https://travis-ci.com/github/matplotlib/matplotlib/builds/158347125 <- last good build on master
    https://travis-ci.com/github/matplotlib/matplotlib/builds/158411743 <- first fail on master
    #17025 is that merge and it passed CI

    @QuLogic:

    I think that was skipped in the 'working' build

    SKIPPED [1] lib/matplotlib/tests/test_bbox_tight.py:83: Cannot compare pdf files on this system

    @tacaswell:

    interesting...

    @QuLogic

    and a diff of the log says that ghostscript is now installed as a dependency of imagemagick
    oh, and 1288 skipped vs 678 skipped

    @jklymak:


    Same problem I was getting before
    I upgraded ghostscript and the above is what the new failed diff looks like

    @QuLogic:

    ah, so it's new ghostscript that does it? what versions?

    @jklymak:

    Sorry yes new gs. Stepped away from computer for a while so not sure what the versions are

    gs
    GPL Ghostscript 9.52 (2020-03-19)
    

    is the new one.

    $ gs GPL Ghostscript 9.26 (2018-11-20) was my old one

  4. QuLogic commented on Apr 8, 2020

    Member

    I can reproduce in a Fedora 32 container with ghostscript 9.52 , but it only changes the result from 5.534 to 6.457. There must be something else different as well. Scratch that, it was just the test image cache.

  5. added this to the v3.2.2 milestone on Jun 16, 2020
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

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions


      Back | FazBrowse Home | New Git URL