<div dir="ltr"><div>Hi Samuel</div><div><br></div><div>It looks like excellent work. I checked my polar plot and the labels are indeed a little bit off (mostly visible with 3-digit numbers, i.e. 120, 150, 180 and so on degrees). It works OK for now. I think your upgraded polarplot should go into the main tree - the plots as seen on the picture you included looks excellent.</div><div><br></div><div>If overplotting could be fixed, that would also be very nice.</div><div><br></div><div>P.S. it reminds me that Scilab 6 was released about a year ago. Normally such a first release with such big changes will require some bug fixing, but surprisingly this seems not to be the case. Impressive how well 6.0.0 was executed (and it also took a couple of years ...). I could not execute my latest script (with the polar and contour plots) in Scilab 5 -- it ran out of memory.</div><div><br></div><div>I wonder if anyone can say what's up in regard to Scilab releases?</div><div><br></div><div>Best regards,</div><div>Claus</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jan 21, 2018 at 9:41 PM, Samuel Gougeon <span dir="ltr"><<a href="mailto:sgougeon@free.fr" target="_blank">sgougeon@free.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF">
<div class="m_-9031748470153732225moz-cite-prefix">Hello Clauss,<br>
<br>
After a first polarplot upgrade in 2010, i started writting some
other improvements in 2016, first about still improving angular
and radial labels positionning. Yesterday, i took the opportunity
of your message to post on the CodeReview the polarplot.sci in
progress, and to go on a bit, mainly about proper labels for polar
datatips.<br>
You can already use the scripts as is:
<a class="m_-9031748470153732225moz-txt-link-freetext" href="https://codereview.scilab.org/#/c/19684/" target="_blank">https://codereview.scilab.org/<wbr>#/c/19684/</a><br>
Just click on the polarplot.sci link, and then you may download
it, or copy/past the content.<br>
<br>
The trick with gca().rotation_angles reminded y Rafael is well
known, but due to labels uncentered on their anchor, using it
currently shifts labels (see the screenshot below). This is fixed
in the CodeReview version.<br>
<br>
If you want to be fully back-compatible with this new
polarplot.sci, you should simply comment lines<br>
performing some glue(). This won't prevent other improvements
working.<br>
<br>
Other improvements can be proposed, as listed in the review
header:<br>
<ul>
<li> DONE:</li>
<ul>
<li>centering of angular labels: Already allows shifting the
0° position and the angular direction (clockwise/trigo),
using gca().rotation_angles</li>
<li>Optimized positionning of radial labels</li>
<li>Glue into 4 compounds lines of the radial frame lines of
the circular frame angular labels radial labels (not
backcompatible => To be discussed carefully).<br>
</li>
<li>polarplot() canceled external drawlater(): FIXED</li>
<li>Datatips: implement a convenient default display function
<br>
</li>
</ul>
<li>WIP:</li>
<ul>
<li>Scilab Enhancement Proposal</li>
<li><a href="http://bugzilla.scilab.org/5907:" target="_blank">http://bugzilla.scilab.org/<wbr>5907:</a>
enable overplotting</li>
<li><a href="http://bugzilla.scilab.org/5693:" target="_blank">http://bugzilla.scilab.org/<wbr>5693:</a>
page to be overhauled</li>
</ul>
<li>TO BE ANALYZED</li>
<ul>
<li>Angular labels with a rect box disappear or are unclipped
over neighbouting axes.</li>
<li>Enable specifying major values for the radial and angular
frame (to be analyzed)</li>
<li>Enable radial logarithmic scale</li>
<li>Extend replot() to polar plots
</li>
</ul>
</ul>
<p>This may be seen as a "pre-SEP" :). A proper SEP will be
required. To come.<br>
<br>
In my opinion, the trick with gca().rotation_angles is not fully
convenient because</p>
<ul>
<li>it may switch xlabel() and ylabel() positions, what could be
disturbing</li>
<li>radial labels can be turned displayed along a row, what must
be avoided to avoid overlays when labels are quite long.</li>
</ul>
<p>So, the final implementation won't use this trick.</p>
<p>Later, with a priority clearly lower, the same job shall be
done for graypolarplot().</p>
<p>Cheers<br>
Samuel<br>
</p>
</div>
<img width="559" height="377" alt="" src="cid:part3.A1D8B9F6.4793A450@free.fr">
</div>
<br>______________________________<wbr>_________________<br>
users mailing list<br>
<a href="mailto:users@lists.scilab.org">users@lists.scilab.org</a><br>
<a href="http://lists.scilab.org/mailman/listinfo/users" target="_blank" rel="noreferrer">http://lists.scilab.org/<wbr>mailman/listinfo/users</a><br>
<br></blockquote></div><br></div>