Skip to content

return type of "step" function [sf#1] #2

Description

@murrayrm

Reported by thomas_ha on 2011-04-19 18:37 UTC
The function "step" is supposed to return an array for the time:

T, yout = step(sys, T, X0)
T: array
Instead, a matrix is returned. To convert it to a time array, currently the following step is required (which should not be required):
t2 = array(T)[0,:]

Activity

  1. murrayrm commented on Aug 9, 2014

    @murrayrm
    MemberAuthor

    Commented by eike on 2011-06-10 14:48 UTC
    I had the same problem, and I therefore created my own branch of the library. In this branch I fixed this bug, and I also implemented the simulation of MIMO systems.

    Look at:
    https://bitbucket.org/eike_welk/python-control/

    The current revision (193) works IMHO well.

    The easiest way to get the library is with the version control system "Mercurial". You open a terminal window and type:

    hg clone https://eike_welk@bitbucket.org/eike_welk/python-control #download the code
    cd python-control/
    hg update 193 #This is optional - move to revision 193 - I usually check in working code
    su #get administrative privileges - depends on your operating system
    python setup.py install
    rm -r build dist #clean up
    exit

    I would be glad if you tried my version of Python Control. If the instructions are too complicated for you, Ill think of a way, how you can get a regular package.

  2. changed the title [-]return type of "step" function[/-] [+]return type of "step" function [sf#1][/+] on Aug 9, 2014
  3. self-assigned this
    on Aug 9, 2014
  4. murrayrm commented on Aug 9, 2014

    @murrayrm
    MemberAuthor

    Commented by eike on 2011-06-10 14:56 UTC
    The bug tracker garbled my instructions to downloa and install the library, so here are they again:

    hg clone https://eike_welk@bitbucket.org/eike_welk/python-control
    cd python-control/
    hg update 193
    su #depends on your operating system
    python setup.py install
    rm -r build dist
    exit

  5. murrayrm commented on Aug 9, 2014

    @murrayrm
    MemberAuthor

    Commented by murrayrm on 2011-06-22 18:28 UTC
    This is resolved in version 0.4d (current trunk). All time response functions now return arrays instead of matrices. Will close when 0.4d is released.

  6. murrayrm commented on Aug 9, 2014

    @murrayrm
    MemberAuthor

    Commented by sf-robot on 2011-07-06 19:20 UTC
    This Tracker item was closed automatically by the system. It was
    previously set to a Pending status, and the original submitter
    did not respond within 14 days (the time period specified by
    the administrator of this Tracker).

  7. added a commit that references this issue on Aug 11, 2014
  8. added a commit that references this issue on Feb 16, 2025
  9. added a commit that references this issue on Apr 26, 2025
  10. added a commit that references this issue on Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions