Description

murrayrm

opened on Aug 9, 2014

Issue body actions

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

   on Aug 9, 2014

   MemberAuthor

   More actions

   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://[email protected]/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. murrayrm

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

   self-assigned this

   on Aug 9, 2014
4. murrayrm

   closed this as completedon Aug 9, 2014
5. murrayrm commented on Aug 9, 2014

   

   murrayrm

   on Aug 9, 2014

   MemberAuthor

   More actions

   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://[email protected]/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
6. murrayrm commented on Aug 9, 2014

   

   murrayrm

   on Aug 9, 2014

   MemberAuthor

   More actions

   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.
7. murrayrm commented on Aug 9, 2014

   

   murrayrm

   on Aug 9, 2014

   MemberAuthor

   More actions

   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).
8. jgoppert

   added a commit that references this issue on Aug 11, 2014

   Merge pull request python-control#2 from kdavies4/master

   9b02489
9. murrayrm

   added a commit that references this issue on Apr 11, 2019

   Small update to trigger Readthedocs, attempt #2

   2ffb9c8
10. murrayrm

    added a commit that references this issue on Feb 16, 2025

    Merge pull request #2 from murrayrm/lkies-streamplot

    76965e1
11. josiahdelange

    added a commit that references this issue on Apr 26, 2025

    Attempt python-control#2 at 397efab, based on linter recommendation

    cc02712
12. murrayrm

    added a commit that references this issue on Jun 25, 2025

    Disk margin calculations (#1146)

    aa92b65

Issue actions