-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add voronoi grid #30
Comments
So is this a suggestion to add the functionality or to put this in the docs? |
Well I don't know what you can do with that 😄 . Just that voronoi tesselation would be nice to have so I propose to work on it if it's something you think is relevant. |
@virgile-baudrot you see my dependency concerns are widespread in the Julia world ;) But this would be great, but without ArchGDAL. |
Just for clarity - isn't rasterized voronoi just nearest-neighbor from a grid to the set of points? We have nearestneighbors.jl as a dep already. |
I think it's a good improvement if we don't need ArchGDAL - I think @mkborregaard suggestion makes sense! |
Wait, so is it NearestNeighborElement? I've added an option to set various values of k, but for k=1, this is a Voronoi tessellation? |
I was wondering if it wasn't just that yes. But couldn't tell from the code last night - I was heading home late on the train and was beer coding 😅 |
But voronoi is just a vector format nearest neighbor yes |
Oh great. I didn't know that. So fantastic if it's already there! |
Let's be explicit and add a |
ok, I'll try to implement it and PR |
This was closed by #46 |
The following lines provide a simple way to make grid landscape using voronoi tesselation. It's working but need to be reframe to fit package requirement.
Computing the voronoi is done using VoronoiCells.jl and then the rasterization is by using ArchGDAL.jl.
Digging into both dependencies, I should be able to make things clearer.
I created 3 functions. The 2 first create the set of polygones
And the last is making a rasterization providing some features:
The text was updated successfully, but these errors were encountered: