GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — test_bdalg_functions (control.tests.iosys_test.TestIOSys) fails · Issue #389 · python-control/python-control

test_bdalg_functions (control.tests.iosys_test.TestIOSys) fails #389

New issue
Copy link
New issue
Copy link
Closed
Closed
test_bdalg_functions (control.tests.iosys_test.TestIOSys) fails#389
Copy link
Milestone
0.8.4

Description

@flokli
flokli
opened on Apr 7, 2020
Issue body actions

This is with:

  • python 3.7
  • python-control 0.8.3
  • matplotlib 3.1.3
  • numpy 1.18.1
  • scipy 1.4.1

======================================================================
FAIL: test_bdalg_functions (control.tests.iosys_test.TestIOSys)
Test block diagram functions algebra on I/O systems
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/control-0.8.3/control/tests/iosys_test.py", line 457, in test_bdalg_functions
    np.testing.assert_array_almost_equal(ios_y, lin_y, decimal=3)
  File "/nix/store/dh3b3w7lkcy6p56x0fbv17k0k5winx0y-python3.7-numpy-1.18.1/lib/python3.7/site-packages/numpy/testing/_private/utils.py", line 1047, in assert_array_almost_equal
    precision=decimal)
  File "/nix/store/dh3b3w7lkcy6p56x0fbv17k0k5winx0y-python3.7-numpy-1.18.1/lib/python3.7/site-packages/numpy/testing/_private/utils.py", line 846, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Arrays are not almost equal to 3 decimals

Mismatched elements: 5 / 200 (2.5%)
Max absolute difference: 0.00176677
Max relative difference: 0.33094816
 x: array([[ 0.000e+00,  1.055e-01,  2.183e-01,  3.356e-01,  4.551e-01,
         5.746e-01,  6.921e-01,  8.055e-01,  9.131e-01,  1.013e+00,
         1.105e+00,  1.187e+00,  1.257e+00,  1.316e+00,  1.362e+00,...
 y: array([[ 0.000e+00,  1.055e-01,  2.183e-01,  3.357e-01,  4.553e-01,
         5.747e-01,  6.920e-01,  8.053e-01,  9.129e-01,  1.013e+00,
         1.105e+00,  1.187e+00,  1.257e+00,  1.316e+00,  1.362e+00,...

----------------------------------------------------------------------
Ran 463 tests in 19.914s

FAILED (failures=1, errors=2, skipped=118)

Activity

  1. bnavigator commented on Apr 7, 2020

    @bnavigator
    bnavigator
    on Apr 7, 2020
    Contributor
    More actions

    Already fixed by #366, which is merged but of course not in python-control 0.8.3.

  2. flokli commented on Apr 7, 2020

    @flokli
    flokli
    on Apr 7, 2020
    Author
    More actions

    Thanks for the feedback! Any plans on doing a new release containing this?

  3. flokli commented on Apr 7, 2020

    @flokli
    flokli
    on Apr 7, 2020
    Author
    More actions

    I'd also love it due to #380.

  4. murrayrm
    added this to the 0.8.4 milestone on Jul 11, 2020
  5. murrayrm commented on Jul 11, 2020

    @murrayrm
    murrayrm
    on Jul 11, 2020
    Member
    More actions

    Marking this with milestone 0.8.4 to keep track of the fact that it is fixed there. Closing out as an open issue.

  6. murrayrm
    closed this as completedon Jul 11, 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

      • 0.8.4

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402503 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.