(
Jump to the original submission)
Fri 14 Mar 2014 07:49:23 PM UTC, comment #6:
I reworked the fix:
https://bitbucket.org/paramaniac/control/commits/057bd0b4893b6093ad7082f4481daf24906cde9c
|
Mon 10 Mar 2014 09:47:39 PM UTC, comment #5:
This change fixes it for me.
thanks
|
Mon 10 Mar 2014 08:29:01 PM UTC, comment #4:
Thanks for your reports! I've pushed the following changeset:
https://bitbucket.org/paramaniac/control/commits/71e2970cc8adb6773744dbc8f7b07f2f5dc6bad2
It solves the problem for me. Could you please test it and report whether there are any unexpected issues?
Cheers,
Lukas
|
Mon 10 Mar 2014 02:44:47 AM UTC, comment #3:
The problem seems to be in the repeated roots in the numerator
the following code would be a workaround until we get it fixed
n=poly([ -2 -2.0001])
d=[1 8 1 8 0]
roots(n)
roots(d)
ss1=tf(n,d)
ss1
rlocus(ss1);
|
Mon 10 Mar 2014 02:38:28 AM UTC, comment #2:
Confirmed here as well, updating bug description and tags, rlocus is part of the control package.
|
Mon 10 Mar 2014 02:21:09 AM UTC, comment #1:
Confirmed in ubuntu 12.04 and octave 3.8.1-rc3
|
Mon 10 Mar 2014 01:36:52 AM UTC, original submission:
octave:63> rlocus(tf([1 4 4], [1 8 1 8 0]));
warning: division by zero
error: roots: inputs must not contain Inf or NaN
error: called from:
error: /usr/share/octave/3.6.4/m/polynomial/roots.m at line 85, column 5
error: evaluating argument list element number 1
error: evaluating argument list element number 1
error: /usr/share/octave/packages/control-2.4.3/rlocus.m at line 173, column 23
|
(Note: upload size limit is set to 16384 kB, after insertion of the required
escape characters.)
Attach File(s):
Comment:
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
Follow 10 latest changes.