Skip to content

Conversation

@jnnr
Copy link
Owner

@jnnr jnnr commented Nov 12, 2025

Reviewed and refined changes from #16

  • simplified variable names
  • removed unnecessary steps
  • used Gregor's own function for zonal sum: aggregate_raster_to_polygon

TODOs

  • Maybe parallelize aggregate_raster_to_polygon with dask, too?
  • Use the example data (NL) for another test.

@irm-codebase
Copy link

@jnnr just ran this on the powerplants module for MNE, MEX and European nations.
Works like a charm.

You can see the moment dask takes over for the European case. Both swap and RAM get close to the max, but never cause a catastrophic failure.
image

I did not run numerical tests on the produced raster, but an eyeball tests suggests it's good (the data range is the same as in #16 (comment))
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants