Skip to content
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

Abstract methods for creating catchment indicators #346

Open
carlhiggs opened this issue Jul 21, 2023 · 0 comments
Open

Abstract methods for creating catchment indicators #346

carlhiggs opened this issue Jul 21, 2023 · 0 comments

Comments

@carlhiggs
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
We currently have a method for creating catchment indicators that is more or less hard coded for population density and intersection density per sqkm within 1000m walkable neighbourhood catchments. We use an approximation method to make city-scale analysis for sample points feasible:

  1. preprocessing a population grid (eg 100m resolution) with relevant attributes (e.g. count or density)
  2. evaluate neighbourhood statistics for nodes, by identifying the grid cells reachable within a specified distance (ie. 1000m)
  3. calculate proximity-weighted average of density statistics for sample points

Describe the solution you'd like
The current implementation should be abstracted and made more general to support adding new kinds of catchment indicators (density, following the same pattern, but potentially other kinds too). This ties in with issue #343.

Potentially we should also investigate feasibility of calculating statistics for all sample points directly; given other improvements in process efficiency, that may be feasible now. The challenge remains managing time and memory intensive processing, particularly given uncertainty of computational resources available on users' computers - might be laptop, might be powerful workstation. That would have been the historic rationale for the approximation approach - but always good to revisit these things.

carlhiggs added a commit that referenced this issue Jul 24, 2023
…346) by creating a raster_to_db function that is now used when importing raster population grids. Also added function to drop a single table (#347), and addressed a potential division by zero error for study regions with population < 10000 (#349)
@carlhiggs carlhiggs mentioned this issue Jul 24, 2023
@carlhiggs carlhiggs mentioned this issue Aug 1, 2023
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

No branches or pull requests

1 participant