<div dir="ltr">before that commit, the following code was working:<div><br></div><div> STYLE<br> COLOR 255 255 255<br> SYMBOL "rectangle3"<br> ANGLE -45<br> SIZE 8<br> GAP 10<br> END</div><div><br></div><div>after that commit, it no longer works, as ANGLE -45 is not accepted anymore, instead it had to be changed to <br><div><br></div><div>STYLE<br> COLOR 255 255 255<br> SYMBOL "rectangle3"<br> ANGLE 315<br> SIZE 8<br> GAP 10<br> END<br></div><div><br></div><div><div>the documentation <a href="https://mapserver.org/mapfile/style.html">https://mapserver.org/mapfile/style.html</a> say it must be of type DOUBLE, but double can be negative and nothing say explicitly in the documentation that the value can not be negative.</div></div><div><br></div><div>I see this discussion here:<br><a href="https://github.com/MapServer/MapServer/issues/6463">https://github.com/MapServer/MapServer/issues/6463</a><br></div><div>but no comment if something was done or agreed on about this.</div><div>A note should be added to the documentation to make that clear, as it was done for label size (see below).</div><div>I can do a PR if that would be acceptable ?</div><div><br></div><div>for label size, I see that a note was added saying the value must be greater than zero on 26 january 2022, so I guess this is on me for not noticing it (but im not alone not noticing it so I guess it wasnt clearly stated/announced either)</div><div><br></div><div>Best regards</div><div>Oliver</div><div><br></div><div> </div><div><br></div><div><dl><dd style="margin-top:3px;margin-bottom:0px;margin-left:30px;line-height:1.5em;color:rgb(62,67,73);font-family:Arial,sans-serif;font-size:12.8px"><div><br></div></dd><dd style="margin-top:3px;margin-bottom:0px;margin-left:30px;line-height:1.5em;color:rgb(62,67,73);font-family:Arial,sans-serif;font-size:12.8px"><div><br></div></dd></dl></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 30, 2023 at 1:27 PM Rahkonen Jukka <<a href="mailto:jukka.rahkonen@maanmittauslaitos.fi" target="_blank">jukka.rahkonen@maanmittauslaitos.fi</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>
<div lang="FI">
<div>
<p class="MsoNormal"><span>Hi,<u></u><u></u></span></p>
<p class="MsoNormal"><span><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Maybe the commit could be adjusted in Mapserver 7. Out of curiosity, what is the use case for zero sized labels and symbols? To disable them, or? About the angle, by the Mapserver 7.0
documentation, doesn’t it feel like negative angles were not thought to work ever?<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><br>
“Angle, given in degrees, to rotate the symbol (counter clockwise). Default is 0 (no rotation). If you have an attribute that specifies angles in a clockwise direction (compass direction), you have to adjust the angle attribute values before they reach MapServer
(360-ANGLE), as it is not possible to use a mathematical expression for ANGLE.”<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">-Jukka Rahkonen-<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(225,225,225);padding:3pt 0cm 0cm">
<p class="MsoNormal"><b><span lang="EN-US">Lähettäjä:</span></b><span lang="EN-US"> MapServer-users <<a href="mailto:mapserver-users-bounces@lists.osgeo.org" target="_blank">mapserver-users-bounces@lists.osgeo.org</a>>
<b>Puolesta </b>Oliver Christen<br>
<b>Lähetetty:</b> tiistai 30. toukokuuta 2023 13.59<br>
<b>Vastaanottaja:</b> <a href="mailto:mapserver-users@lists.osgeo.org" target="_blank">mapserver-users@lists.osgeo.org</a><br>
<b>Aihe:</b> [MapServer-users] breaking changes in branch-7-6 after commit of experimental changes from branch-8-0<u></u><u></u></span></p>
</div>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<div>
<div>
<p class="MsoNormal">Dear Mapserver commiters<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">this commit there:<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><a href="https://github.com/MapServer/MapServer/pull/6818" target="_blank">https://github.com/MapServer/MapServer/pull/6818</a><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">broke a lot of things, <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">like style angles not supporting negative values anymore or label size mandatorily greater than zero (also possibly for symbol)<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Im not against changes, but changes should be: <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">a) documented <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">b) announced with preferably a deprecation period<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">so, please revert that commit or provide adequate change/documentation for all the breaking changes.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Thank you<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">Best regards<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">Oliver Christen<br clear="all">
<u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<p class="MsoNormal"><span>-- </span><u></u><u></u></p>
<div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p style="margin:0cm"><a href="https://www.camptocamp.com/en/news-events/camptocamp-flagship-office-bussigny" target="_blank"><b><span style="font-size:18pt;font-family:Arial,sans-serif;color:rgb(17,85,204);border:1pt none windowtext;padding:0cm;text-decoration:none"><img border="0" width="340" height="120" style="width: 3.5416in; height: 1.25in;" id="m_4601368731276966928m_1364736422636054946_x0000_i1025" src="https://lh5.googleusercontent.com/4RpzNqlUvMoqWPtwuUWigz1_dKPkUN4OQOKshy_zixf9Gi6JzQJRjs3oHwxYaiADK7htQjUTodRa-OUr4YEKbgs9SsmqPG6QYjqKj3wbXoQ6snhaqq2eEJTeB1FZHKK7SApWcSZk45Fe3w4grA"></span></b></a><span style="color:rgb(34,34,34)"><u></u><u></u></span></p>
<p style="margin:0cm"><b><span style="font-size:9pt;font-family:Arial,sans-serif;color:rgb(255,104,10)">New address from 1st September 2022</span></b><span style="color:rgb(34,34,34)"><u></u><u></u></span></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p style="margin:0cm"><b><span style="font-size:9pt;font-family:Arial,sans-serif;color:rgb(122,127,130)">Camptocamp SA</span></b><span style="color:rgb(34,34,34)"><u></u><u></u></span></p>
<p style="margin:0cm"><span style="font-size:9pt;font-family:Arial,sans-serif;color:rgb(122,127,130)">Route de la Chaux 4</span><span style="color:rgb(34,34,34)"><u></u><u></u></span></p>
<p style="margin:0cm"><span style="font-size:9pt;font-family:Arial,sans-serif;color:rgb(122,127,130)">1030 Bussigny</span><span style="color:rgb(34,34,34)"><u></u><u></u></span></p>
<p style="margin:0cm"><span style="font-size:9pt;font-family:Arial,sans-serif;color:rgb(122,127,130)">Suisse</span><span style="color:rgb(34,34,34)"><u></u><u></u></span></p>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div></blockquote></div>