[GRASS-dev] GSoC, reimplementation of modules v.voronoi and v.delaunay

[Wolf Bergenheim]

Hi,

Ehh.. I easily make the mistake of replying directly. I'm bringing this 
back to the list, since I feel it will interest all of the community.


On 24.04.2008 10:54, Martin Pavlovsky wrote:
> Thank you for the warm welcome.
> 
> I think that having only one module has many advantages in different 
> perspectives. Since VD and DT are so closely related, I presume that 
> more than 70% of the code would be very similar for both modules if they 
> were separated. This replication would be avoided by joining them 
> together. The relationship between VD and DT would be more obvious for 
> the user if there was a single module named "v.voronoi_delaunay" or sth 
> similar. User will choose VD or DT by setting parameters, it seems to me 
> as the most natural way. This DT -> VD and VD -> DT conversion is very 
> important feature which is lacking in current implementation, I am all 
> for it. One disadvantage of having only one module might be slightly 
> steeper learning curve, but I don't see it as a major obstacle.

I agree. It is better to have it as one module. I wonder what would be a 
good name. v.voronoi.delauny seems a bit awkward but very obvious. I 
can't think of a better name. Anybody else have some idea?

Another way to handle it would be to make two modules, but re-use that 
70% by having it in a library or something.

--Wolf

> 
> On Thu, Apr 24, 2008 at 5:57 AM, Wolf Bergenheim 
> <wolf+grass at bergenheim.net <mailto:wolf%2Bgrass at bergenheim.net>> wrote:
> 
> 
> 
>     Welcome! And congrats once again. I hope you'll have as much fun
>     here as I do!
> 
>     Do you plan on making two modules (like we have now), or one one
>     single module that can create either a DT or VD depending on switches?
> 
>     One thing which you talk about, and it would be cool to have. Given
>     either a DT or VD, it would be cool to be able to use the module to
>     convert it to the other, since they are the duals of each other.
> 
>     --Wolf
> 
>     -- 
> 
>     <:3 )---- Wolf Bergenheim ----( 8:>
> 
> 

-- 

<:3 )---- Wolf Bergenheim ----( 8:>