6:39 a.m.
Hi,
Given Allin's announcement that a new version will be released soon, I
ran the test scripts.
1. For the following script the assertion failed on the latest git-version:
https://sourceforge.net/p/gretl/workspace/ci/master/tree/tests/test_scrip...
I am pretty sure that I've never seen it failing before.
2. This gridplot example
https://sourceforge.net/p/gretl/workspace/ci/master/tree/tests/test_scrip...
also fails with the message:
<error>
*** error in function multiplot, line 32
gnuplot --input="/home/artur/.gretl/outfile.3GVZ71"
--output="display"
called by function plot_season_plot
WARNING: Failed to execute file 'gridplot_seasonplot.inp'. Ignore.
</error>
Best
Artur
6:53 a.m.
On Wed, 12 Jul 2023, Artur T. wrote:
Hi,
Given Allin's announcement that a new version will be released soon, I ran
the test scripts.
1. For the following script the assertion failed on the latest git-version:
https://sourceforge.net/p/gretl/workspace/ci/master/tree/tests/test_scrip...
I am pretty sure that I've never seen it failing before.
It's the "sumc" function, which fails for complex arguments. Try this:
<hansl>
set seed 39879
X = mnormal(3,3)
l = eigen(X)
print l
eval sumc(diag(X))
eval sumc(l)
</hansl>
which gives
<output>
? print l
l (3 x 1)
-1.19391 + 0.00000i
0.73406 + 0.96306i
0.73406 - 0.96306i
? eval sumc(diag(X))
0.27420
? eval sumc(l)
1.27420 + 0.00000i
</output>
I guess this must be related to the new "skip_na" optional flag, but I
have no time to check this morning, sorry.
-------------------------------------------------------
Riccardo (Jack) Lucchetti
Dipartimento di Scienze Economiche e Sociali (DiSES)
Università Politecnica delle Marche
(formerly known as Università di Ancona)
r.lucchetti(a)univpm.it
http://www2.econ.univpm.it/servizi/hpp/lucchetti
-------------------------------------------------------
12:05 p.m.
On Wed, Jul 12, 2023 at 2:53 AM Riccardo (Jack) Lucchetti
<p002264(a)staff.univpm.it> wrote:
On Wed, 12 Jul 2023, Artur T. wrote:
> Hi,
>
> Given Allin's announcement that a new version will be released soon, I ran
> the test scripts.
>
> 1. For the following script the assertion failed on the latest git-version:
>
>
https://sourceforge.net/p/gretl/workspace/ci/master/tree/tests/test_scrip...
>
> I am pretty sure that I've never seen it failing before.
It's the "sumc" function, which fails for complex arguments. [...]
Indeed. This was due to a long-standing error in
gretl_cmatrix_vector_stat() -- a typo, basically. That's now fixed in
git.
Allin
12:41 p.m.
Am 12.07.2023 um 14:05 schrieb Cottrell, Allin:
On Wed, Jul 12, 2023 at 2:53 AM Riccardo (Jack) Lucchetti
<p002264(a)staff.univpm.it> wrote:
> On Wed, 12 Jul 2023, Artur T. wrote:
>> Hi,
>>
>> Given Allin's announcement that a new version will be released soon, I ran
>> the test scripts.
>>
>> 1. For the following script the assertion failed on the latest git-version:
>>
> It's the "sumc" function, which fails for complex arguments. [...]
Indeed. This was due to a long-standing error in
gretl_cmatrix_vector_stat() -- a typo, basically. That's now fixed in
git.
In any case, an excellent example of the usefulness of Artur's unit
testing efforts!
thanks
sven
7:58 a.m.
Am 12.07.23 um 14:41 schrieb Sven Schreiber:
Am 12.07.2023 um 14:05 schrieb Cottrell, Allin:
> On Wed, Jul 12, 2023 at 2:53 AM Riccardo (Jack) Lucchetti
> <p002264(a)staff.univpm.it> wrote:
>> On Wed, 12 Jul 2023, Artur T. wrote:
>>> Hi,
>>>
>>> Given Allin's announcement that a new version will be released soon,
>>> I ran
>>> the test scripts.
>>>
>>> 1. For the following script the assertion failed on the latest
>>> git-version:
>>>
>> It's the "sumc" function, which fails for complex arguments. [...]
> Indeed. This was due to a long-standing error in
> gretl_cmatrix_vector_stat() -- a typo, basically. That's now fixed in
> git.
Thanks for the quick fix, Allin. Confirmed it works now as expected.
In any case, an excellent example of the usefulness of Artur's
unit
testing efforts!
Yes, I am glad that it helps to detect problems.
Best
Artur
1:04 p.m.
On Wed, Jul 12, 2023 at 2:39 AM Artur T. <atecon(a)posteo.de> wrote:
Hi,
Given Allin's announcement that a new version will be released soon, I
ran the test scripts.
1. For the following script the assertion failed on the latest git-version:
https://sourceforge.net/p/gretl/workspace/ci/master/tree/tests/test_scrip...
I am pretty sure that I've never seen it failing before.
As I mentioned in response to Jack's comment, that's now fixed.
The bug has been there for a long time, but since the test script uses
random matrices and doesn't set a seed, you could get lucky and have
eigenvalues with zero imaginary parts, in which case the script
wouldn't fail.
2. This gridplot example
https://sourceforge.net/p/gretl/workspace/ci/master/tree/tests/test_scrip...
also fails with the message:
<error>
*** error in function multiplot, line 32
> gnuplot --input="/home/artur/.gretl/outfile.3GVZ71"
--output="display"
called by function plot_season_plot
WARNING: Failed to execute file 'gridplot_seasonplot.inp'. Ignore.
</error>
That's inevitable I'm afraid. season_plot produces something that's
already a multiplot, and that cannot be handled by gpbuild (since
multiplots cannot be nested in gnuplot).
However... there's a fixable issue in the season_plot package which
would help. The package produces two plot components labeled
"obsmajor" and "obsminor" which can be plotted individually or
jointly. The joint plot is inherently a "multiplot" instance and so
can't work with gpbuild. The individual plots are NOT inherently
"multi" but the package writes them as if they were: the gnuplot
commands generated by season_plot look like this:
set multiplot layout 1,1 rowsfirst
# one individual plot
reset
unset multiplot
If the "set multiplot ", "reset" and "unset multiplot" lines
were
simply omitted, then a set of such plots could be handled by gpbuild.
Allin
528
days inactive
529
days old
5 comments
4 participants
participants (4)
-
Artur T. -
Cottrell, Allin -
Riccardo (Jack) Lucchetti -
Sven Schreiber