<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">Le 22/01/2019 à 17:56, Samuel Gougeon a
écrit :<br>
</div>
<blockquote cite="mid:a05f47e0-447d-30f5-d676-06053b369ae8@free.fr"
type="cite">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<div class="moz-cite-prefix">Le 22/01/2019 à 17:40, Stéphane
Mottelet a écrit :<br>
</div>
<blockquote cite="mid:e1000819-ebb2-fd9a-eff3-156a6ebf3e43@utc.fr"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
<div class="moz-cite-prefix">Le 22/01/2019 à 17:06, Samuel
Gougeon a écrit :<br>
</div>
<blockquote type="cite"
cite="mid:8492f083-f5d0-c80f-ea79-1dd9d78baecd@free.fr">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<div class="moz-cite-prefix">Le 22/01/2019 à 16:34, Stéphane
Mottelet a écrit :<br>
</div>
<blockquote
cite="mid:9bfed43f-66ed-a93f-61cf-d2922fa0cd24@utc.fr"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
<div class="moz-cite-prefix">Le 22/01/2019 à 16:25, Samuel
Gougeon a écrit :<br>
</div>
<blockquote type="cite"
cite="mid:922fed48-f7af-2d09-e468-e198310ba0af@free.fr">Hello
Stéphane, <br>
<br>
Le 17/01/2019 à 08:37, Stéphane Mottelet a écrit : <br>
<blockquote type="cite">Hello Samuel, <br>
<br>
I have initially voted for this behavior (sparse([]) ==
( 0, 0) zero sparse matrix ) but your remark on
numerous occurences of "sparse([])" in scilab prevented
to do so. <br>
</blockquote>
<br>
Why? My remark aimed to bring the attention to existing
occurrences and to prevent merging the commit without
updating them, not to prevent updating them. <br>
Then these occurrences changed your opinion and vote, not
mine. <br>
<br>
Best regards <br>
Samuel <br>
</blockquote>
<p>Please see comment #8:</p>
<p>Anyway, <b>as bug #15758 is not related to sparse([])
being not sparse</b>, and considered the numerous side
effects, I will restore sparse([]) == double empty matrix.
<b>If you find necessary to have sparse([]) == sparse
empty matrix, please file a bug/whish on BZ.</b></p>
</blockquote>
<br>
Yes, and neither you -- that voted for it -- nor me did it.<br>
And finally we agree: keeping sparse([])==[] was your
decision.<br>
Changing this could be done later if it proves to be better.<br>
For the time being, the situation is the following:<br>
<br>
<b>Scilab 6.0.1</b>:<br>
<tt>--> sparse([])</tt><tt><br>
</tt><tt> ans =</tt><tt><br>
</tt><tt> []</tt><tt><br>
</tt><tt><br>
</tt><tt>--> sparse([],[])</tt><tt><br>
</tt><tt> ans =</tt><tt><br>
</tt><tt>( 0, 0) zero sparse matrix</tt><tt><br>
</tt><tt><br>
</tt><tt>--> sparse([],[],[0,2])</tt><tt><br>
</tt><tt> ans =</tt><tt><br>
</tt><tt> []</tt><br>
<br>
<b>Scilab 6.0.2-</b> after <a class="moz-txt-link-freetext"
href="https://antispam.utc.fr/proxy/2/c3RlcGhhbmUubW90dGVsZXRAdXRjLmZy/codereview.scilab.org/20492"
moz-do-not-send="true">https://codereview.scilab.org/20492</a>
:<br>
<tt>--> sparse([])</tt><tt> // unchanged<br>
</tt><tt> ans =</tt><tt><br>
</tt><tt> []</tt><tt><br>
</tt><tt><br>
</tt><tt>--> sparse([],[])</tt><tt> // unchanged<br>
</tt><tt> ans =</tt><tt><br>
</tt><tt>( 0, 0) zero sparse matrix</tt><tt><br>
</tt><tt><br>
</tt><tt>--> sparse([],[],[0,2])</tt><tt> // <b>CHANGED</b><br>
</tt><tt> ans =</tt><tt><br>
</tt><tt>( 0, 0) zero sparse matrix</tt><br>
<br>
while i don't think that this last change is intentional.<br>
</blockquote>
<p>it is. To me, <br>
</p>
<p><tt>sparse([],[]) <=> sparse([],[],[0,0]) by getting
dims implicitely defined in first two arguments (ij,v), so
explicit size definition in sparse([],[],[2,0]) should give
the same output </tt></p>
</blockquote>
<br>
<tt>Yes<br>
<br>
</tt>
<blockquote cite="mid:e1000819-ebb2-fd9a-eff3-156a6ebf3e43@utc.fr"
type="cite">
<p><tt>(2x0 == empty matrix);</tt></p>
</blockquote>
<br>
<tt>No. Since both other syntaxes yielded [], [] should be
returned. What would make a consistent answer for <b>all</b>
empty inputs.<br>
</tt></blockquote>
<br>
<tt>I mean: Changing<br>
<br>
</tt><tt>--> sparse([],[]) <br>
ans =<br>
( 0, 0) zero sparse matrix<br>
<br>
into<br>
<br>
--> sparse([],[]) <br>
ans =<br>
[]<br>
<br>
</tt>
</body>
</html>