<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">Le 26/11/2016 16:14, Samuel Gougeon a
écrit :<br>
</div>
<blockquote cite="mid:5839A6BF.4050400@free.fr" type="cite">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<div class="moz-cite-prefix">Le 26/11/2016 08:52, Antoine
Monmayrant a écrit :<br>
</div>
.../...
<blockquote cite="mid:3100-58393f00-2d-4cc6b400@100718258"
type="cite">
<pre wrap="">Two issues:
1) the help pages are missing description of most of the parameters,
</pre>
</blockquote>
Parameters description is not missing. It is just written for
developers that are not users ;):<br>
Format and datatype of the parameter, that's all. No need to know
what it represents. Any such need?<br>
I am jocking, but by the way most of pages are written in this
way. Most of pages need a full overhaul by advanced users.<br>
As a contributor, you are welcome, most likely because you know
how much time you loose because of a poor documentation, vs how
much time you would save + using a good doc - contributing to
write it.<br>
<br>
<blockquote cite="mid:3100-58393f00-2d-4cc6b400@100718258"
type="cite">
<pre wrap="">2) the two functions are apparently not using the same definition of frect:
>
> plot2d();
> [wrect,frect,logflag,arect] = xgetech():
> xsetech(wrect, frect)// does not work for the scale part, ie frect
> at line 65 of function xsetech ( /pathtoscilab/share/scilab/modules/graphics/macros/xsetech.sci line 79 )
> Error : Min and Max values for one axis do not verify Min <= Max.
> xsetech(wrect, [frect(1),frect(3),frect(2),frect(4)]) // works fine, scale un changed
</pre>
</blockquote>
I don't see that, for instance using the examples. Could you
please provide a clear example setting parameters with xsetech()
and reading them back from xgetech(), that would not match?
Thanks!<br>
</blockquote>
<br>
<br>
Arg, ok, sorry for misreading your example.<br>
<tt><br>
help xgetech() </tt>tells: <code><br>
"The rectangle [xmin, ymin, xmax, ymax] given by frect.." <br>
while it actually returns [xmin xmax ymin ymax] that matches
gca().data_bounds<br>
<br>
help xsetech tells: "frect = [xmin, ymin, xmax, ymax]"<br>
and that's the way it works.<br>
<br>
The bug was introduced in xgetech() in 2012 when xgetech() became
a macro instead of formerly a builtin function.<br>
This shows that <br>
</code>
<ul>
<li><code>xgetech() is very few used. Otherwise, the bug would
have been spotted before.</code></li>
<li><code>frect from the old xgetech() did not match
gca().data_bounds. This was cryptic.</code></li>
<li><code>xsetech() does not accept gca().data_bounds as frect, but
a puzzled version of it. It is a cryptic usage.</code></li>
</ul>
<p><code>So, here is a <b>Scilab Enhancement Proposal (SEP)</b>:<br>
</code></p>
<ul>
<li><code>xgetech:</code></li>
<ul>
<li><code>fix the code of xgetech.sci in order to match the old
xgetech() version</code></li>
<li><code>set xgetech as obsolete, but keep its code (for
back-compatibility)<br>
</code></li>
<li><code>undocument it</code><br>
</li>
</ul>
<li>xsetech:</li>
<ul>
<li>extend subplot() to replace xsetech(). <br>
Additional subplot() syntaxes to be designed. The new input
formats should be the ones of the unpuzzled gca(). attributes</li>
<li>undocument xsetech() to make it a private internal function.</li>
</ul>
</ul>
<p>What do you think about that?</p>
<p>Samuel<br>
<br>
</p>
</body>
</html>