<div dir="ltr"><div>+ 1<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 20, 2013 at 4:32 PM, GRASS GIS <span dir="ltr"><<a href="mailto:trac@osgeo.org" target="_blank">trac@osgeo.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">#1838: r.mask: allow use of vector map as input<br>
--------------------------+-------------------------------------------------<br>
  Reporter:  mlennert     |       Owner:  grass-dev@…<br>
      Type:  enhancement  |      Status:  closed<br>
  Priority:  normal       |   Milestone:  7.0.0<br>
 Component:  Raster       |     Version:  svn-trunk<br>
Resolution:  fixed        |    Keywords:  mask, vector, r.mask<br>
  Platform:  Unspecified  |         Cpu:  Unspecified<br>
--------------------------+-------------------------------------------------<br>
<br>
</div>Comment(by hamish):<br>
<br>
 To be honest, I find it hard to justify not running `v.to.rast` + r.mask<br>
 as a two step process. You either respect the current region setting, or<br>
 the v.to.rast operation is partially undefined. Having the module change<br>
 the region (and if so, internally, via WIND_OVERRIDE & the<br>
 grass.use_temp_region() python function) for any reason seems like a<br>
 slight misfeature to me.<br>
<br>
 Consider the case that the vector is much larger than the raster (national<br>
 coastline, I'm offen making coastal "land.mask" rasters from that), you<br>
 wouldn't want to rasterize the entire country at the current raster<br>
 resolution. Of course it's possible to only shrink the region with a<br>
 little scripting work, and it could be a nice efficiency trick to do that,<br>
 but be careful that the logic for it might fail for lat/lon near the<br>
 dateline)<br>
<br>
<br>
 do one thing well,<br>
 Hamish<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Ticket URL: <<a href="https://trac.osgeo.org/grass/ticket/1838#comment:8" target="_blank">https://trac.osgeo.org/grass/ticket/1838#comment:8</a>><br>
GRASS GIS <<a href="http://grass.osgeo.org" target="_blank">http://grass.osgeo.org</a>><br>
<br>
</font></span></blockquote></div><br></div>