[GRASS-dev] GRASS 7: replace r.los with r.viewshed?

Hi,

how about replacing r.los with r.viewshed?

Markus

2011/12/1 Markus Neteler <neteler@osgeo.org>:

how about replacing r.los with r.viewshed?

+1

Martin

--
Martin Landa <landa.martin gmail.com> * http://geo.fsv.cvut.cz/~landa

Markus Neteler wrote:

how about replacing r.los with r.viewshed?

I'm supportive of that, starting with trunk. (too
late for 6.4.2) r.los just doesn't scale.

still one final small detail with the color rules
I'd like to see:
  https://trac.osgeo.org/grass/ticket/390

It requires knowing the range of angle values once
0 and 180 have been taken out.

thanks,
Hamish

+1 from me, looking forward to using r.viewshed on some massive maps-
would be nice to have it in the standard install of GRASS.

Dylan

On Thu, Dec 1, 2011 at 12:52 PM, Hamish <hamish_b@yahoo.com> wrote:

Markus Neteler wrote:

how about replacing r.los with r.viewshed?

I'm supportive of that, starting with trunk. (too
late for 6.4.2) r.los just doesn't scale.

still one final small detail with the color rules
I'd like to see:
https://trac.osgeo.org/grass/ticket/390

It requires knowing the range of angle values once
0 and 180 have been taken out.

thanks,
Hamish
_______________________________________________
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Markus Neteler wrote:

how about replacing r.los with r.viewshed?

needs fixing:

grass65/dist.i686-pc-linux-gnu/include/grass/iostream/replacementHeapBlock.h:146: warning: 'str' may be used uninitialized in this function

(3 times).

Hamish

Hamish wrote:

Markus Neteler wrote:

how about replacing r.los with r.viewshed?

needs fixing:

grass65/dist.i686-pc-linux-gnu/include/grass/iostream/replacementHeapBlock.h:146: warning: 'str' may be used uninitialized in this function

(3 times).

Did you ever have a look at the compiler warnings for r.terraflow?

I have already ported r.viewshed to trunk in my local copy, I just
need to submit it...

Markus M