<html>
<head>
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">Le 22/01/2019 à 17:58, Samuel Gougeon a
écrit :<br>
</div>
<blockquote type="cite"
cite="mid:c02f5673-ed38-9dec-b3f6-b14366d4e23b@free.fr">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<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/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>
</tt></blockquote>
<p><tt>I do not agree. I will prepare a patch with </tt><tt>sparse([])
== </tt><tt>(0,0) zero sparse matrix for master and this time I
will track all possible glitches. Every output of sparse(...),
whatever the arguments, should be sparse, even if empty.</tt></p>
<p><tt>S.<br>
</tt></p>
<p><tt><br>
</tt></p>
<blockquote type="cite"
cite="mid:c02f5673-ed38-9dec-b3f6-b14366d4e23b@free.fr"><tt> <br>
</tt> <br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:dev@lists.scilab.org">dev@lists.scilab.org</a>
<a class="moz-txt-link-freetext" href="https://antispam.utc.fr/proxy/1/c3RlcGhhbmUubW90dGVsZXRAdXRjLmZy/lists.scilab.org/mailman/listinfo/dev">https://antispam.utc.fr/proxy/1/c3RlcGhhbmUubW90dGVsZXRAdXRjLmZy/lists.scilab.org/mailman/listinfo/dev</a>
</pre>
</blockquote>
<p><br>
</p>
<pre class="moz-signature" cols="72">--
Stéphane Mottelet
Ingénieur de recherche
EA 4297 Transformations Intégrées de la Matière Renouvelable
Département Génie des Procédés Industriels
Sorbonne Universités - Université de Technologie de Compiègne
CS 60319, 60203 Compiègne cedex
Tel : +33(0)344234688
<a class="moz-txt-link-freetext" href="http://www.utc.fr/~mottelet">http://www.utc.fr/~mottelet</a></pre>
</body>
</html>