Details

    • Type: Bug
    • Status: Resolved
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 1.1.0
    • Fix Version/s: 1.2.0
    • Component/s: visualization
    • Labels:
      None

      Description

      Reported on the mailing list:

      $ ./test.sh
      ...
      
      ======================================================================
      FAIL: test_odd_nice_intervals
      (ocw.tests.test_plotter.TestNiceIntervalsFunction)
      ----------------------------------------------------------------------
      Traceback (most recent call last):
         File
      "/home/toddg/projects/climate/climate/ocw/tests/test_plotter.py", line
      46, in test_odd_nice_intervals
           np.testing.assert_array_equal(result, expected_array)
         File
      "/opt/anaconda3/envs/python27/lib/python2.7/site-packages/numpy/testing/utils.py",
      line 871, in assert_array_equal
           verbose=verbose, header='Arrays are not equal')
         File
      "/opt/anaconda3/envs/python27/lib/python2.7/site-packages/numpy/testing/utils.py",
      line 796, in assert_array_compare
           raise AssertionError(msg)
      AssertionError:
      Arrays are not equal
      
      (mismatch 80.0%)
        x: array([-1.6, -0.8,  0. ,  0.8,  1.6])
        y: array([-2., -1.,  0.,  1.,  2.])
      
      ----------------------------------------------------------------------
      Ran 206 tests in 126.232s
      
      FAILED (failures=1)
      

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user agoodm opened a pull request:

          https://github.com/apache/climate/pull/426

          CLIMATE-895 Make plotter compatible with matplotlib 2.0

          The title is a slight misnomer, what's actually being done is making sure that plotting behavior is more consistent between user-installed matplotlib versions.

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/agoodm/climate CLIMATE-895

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/climate/pull/426.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #426


          commit bd1b8edef58b73abfaf873d46bebf125b8fbc8c5
          Author: Alex Goodman <agoodm@users.noreply.github.com>
          Date: 2017-02-23T00:11:15Z

          Make automatic contour intervals consistent between matplotlib versions


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user agoodm opened a pull request: https://github.com/apache/climate/pull/426 CLIMATE-895 Make plotter compatible with matplotlib 2.0 The title is a slight misnomer, what's actually being done is making sure that plotting behavior is more consistent between user-installed matplotlib versions. You can merge this pull request into a Git repository by running: $ git pull https://github.com/agoodm/climate CLIMATE-895 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/climate/pull/426.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #426 commit bd1b8edef58b73abfaf873d46bebf125b8fbc8c5 Author: Alex Goodman <agoodm@users.noreply.github.com> Date: 2017-02-23T00:11:15Z Make automatic contour intervals consistent between matplotlib versions
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/climate/pull/426

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/climate/pull/426
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user agoodm commented on the issue:

          https://github.com/apache/climate/pull/426

          Seems like travis is stalled on Apache's end... Relevant tests were passing on my end:

          https://travis-ci.org/agoodm/climate/jobs/204421839

          Seems like there are more python 3 errors to fix with the latest versions of numpy though...

          Show
          githubbot ASF GitHub Bot added a comment - Github user agoodm commented on the issue: https://github.com/apache/climate/pull/426 Seems like travis is stalled on Apache's end... Relevant tests were passing on my end: https://travis-ci.org/agoodm/climate/jobs/204421839 Seems like there are more python 3 errors to fix with the latest versions of numpy though...

            People

            • Assignee:
              agoodman Alex Goodman
              Reporter:
              agoodman Alex Goodman
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development