<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 19, 2014 at 3:31 PM, Markus Neteler <span dir="ltr"><<a href="mailto:neteler@osgeo.org" target="_blank">neteler@osgeo.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Anna,<br>
<span class=""><br>
On Tue, Nov 18, 2014 at 5:19 AM, Anna Petrášová <<a href="mailto:kratochanna@gmail.com">kratochanna@gmail.com</a>> wrote:<br>
> Hi all,<br>
><br>
> I added raster digitizer in r62792 in trunk.<br>
<br>
</span>what a great job!<br>
<br>
If you didn't find it: it is in the "Map Display" windows, then<br>
selector on the right side (2D/3D/Vector digitizer/Raster digitizer).<br></blockquote><div><br></div><div>Once everything works, I will probably make a short video tutorial.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> You are welcome to test it and<br>
> report bugs/enhancements.<br>
> It currently allows to draw areas, lines and points and specify width to<br>
> buffer individual features. We can discuss some changes in gui interface and<br>
> behavior if you find the current one intuitive.  There is still a lot to<br>
> improve especially in the drawing part (needs some refactoring of the old<br>
> drawing code).<br>
<br>
</span>One general thing I found (also valid for the vector digitizer): I<br>
wonder where to put a message what the mouse buttons mean.<br>
Maybe in the status bar at bottom of the display/digitizer window?<br>
<span class=""><br>
> Features:<br>
> - draw area, line, point<br>
<br>
</span>... works.<br>
<span class=""><br>
> - specify buffer width for individual features to create for example<br>
> circles, roads of certain width...<br>
<br>
</span>... maybe rename from "Width" to "Buffer width"? Otherwise it could be<br>
confused with the drawing line width.<br></blockquote><div><br></div><div>There is not much space in toolbar..., especially if we in the future add more items (settings, label editing for example). What about just 'Buffer'? Currently the width is the width of the buffered line/diameter of the point (circle), which means 2 x buffer distance, so if we change it to buffer, then the value should be half of the line width?</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> - specify background map<br>
<br>
... works (maybe have a button to optionally take computational region<br>
from that map?)<br></blockquote><div><br></div><div>Yes possibly, I am not sure about the region in general, if there should be any special handling. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> - simple undo<br>
> - save button to save results during drawing<br>
<br>
</span>here I got an issue somewhere:<br>
<br>
Exception in thread Thread-4:<br>
Traceback (most recent call last):<br>
  File "/usr/lib64/python2.7/threading.py", line 811, in<br>
__bootstrap_inner<br>
    self.run()<br>
  File "/home/neteler/software/grass71/dist.x86_64-unknown-<br>
linux-gnu/gui/wxpython/core/gthread.py", line 94, in run<br>
    ret = vars()['callable'](*args, **kwds)<br>
  File "/home/neteler/software/grass71/dist.x86_64-unknown-<br>
linux-gnu/gui/wxpython/rdigit/controller.py", line 432, in<br>
_exportRaster<br>
    if item.GetPropertyVal('widthValue') and \<br>
  File "/home/neteler/software/grass71/dist.x86_64-unknown-<br>
linux-gnu/gui/wxpython/mapwin/graphics.py", line 439, in<br>
GetPropertyVal<br>
    raise KeyError(_("Property does not exist: %s") %<br>
(propName))<br>
KeyError: 'Property does not exist: widthValue'<br>
<span class=""><br>
<br></span></blockquote><div><br></div><div>it normally works... Does this happen every time?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> - keeps drawing order in the output raster map<br>
> - change color of drawing<br>
> - doesn't block gui when exporting raster (which can take some time)<br>
<br>
</span>Nice!<br>
<span class=""><br>
> Does not support (yet?):<br>
> - adding labels<br>
> - interactive setting color of raster cells (not planned, there are other tools)<br>
> - vector export/import<br>
><br>
> Known issues:<br>
> - r.in.poly supports only CELL (I just realized that, this must be changed)<br>
<br>
</span>(meanwhile you changed that already, thanks)<br>
<span class=""><br>
> - various small drawing issues<br>
> - slow when exporting features with buffers (r.grow is slow, probably<br>
> because it's a script)<br>
<br>
</span>Not sure but r.buffer would be appropriate/faster?<br></blockquote><div><br></div><div>I considered that but the cells grown by r.buffer have different values than the original ones, so I would have to reclassify it, so I am not sure if it would be faster in the end.</div><div><br></div><div><br></div><div>Thanks for your feedback!</div><div><br></div><div>Anna</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
Markus<br>
</font></span></blockquote></div><br></div></div>