Re: Octave Control package bug ?

On Wed, Sep 16, 2020 at 12:35 PM Thomas D. Dean <[email protected]>
wrote:

>
> >> On Wed, Sep 16, 2020 at 5:44 AM Andreas Weber <[email protected][1]>
> wrote:
> >
> >
> > Am 31.08.20 um 10:12 schrieb Parameshwara Bhat:
> >> This error I receive even if I give the exact demo commands for the
> functions from the Documentation,like
> >>
> >> "s=tf('s');
> >>   g=1/(s+1);
> >>   pzmap(g); "
> >
> > I can reproduce this with control-3.2.0 and get:
> >
> > octave:1> pkg load control
> > octave:2> s=tf('s');
> > octave:3> g=1/(s+1);
> > octave:4> pzmap(g);
> > error: __zero__: function called with too many inputs
> > error: called from
> >      __zero__
> >      zero at line 140 column 23
> >      pzmap at line 78 column 7
> >
> > This file where this is triggered is @lti/zero.m
> > but I'm not familiar enough to understand that this should be called.
> >
> > -- Andy
> >
>
> I do not see the problem with control 3.2.0.
>
>
>
> octave:10> pkg load control
> octave:11> clear
> octave:12> s=tf('s');
> octave:13> g=1/(s+1);
> octave:14> pzmap(g);
> ans = -7.7682
> octave:15> version()
> ans = 6.0.90
> octave:16> pkg list control
> Package Name  | Version | Installation directory
> --------------+---------+-----------------------
>       control *|   3.2.0 | /home/tomdean/octave/control-3.2.0
>
> Tom Dean
>

Also works for me with a recent hg update

 octave:2> pkg load control
octave:3> s=tf('s');
octave:4> g=1/(s+1);
octave:5> pzmap(g);
octave:6> pkg list
Package Name  | Version | Installation directory
--------------+---------+-----------------------
     control *|   3.2.0 | /home/dima/octave/control-3.2.0
octave:7> version()
ans = 6.0.91
octave:8> __octave_config_info__ ("hg_id")
ans = b12ff2cea46a

control package was rebuilt against this particular version of octave.

I see that there are three different __zero__:
control-3.2.0/@frd/__zero__.m
control-3.2.0/@ss/__zero__.m
control-3.2.0/@tf/__zero__.m

Perhaps loading the wrong one prior to executing the code causes problem
due to caching?

Dmitri.
--