search

alarm-redist / redist

Simulation methods for legislative redistricting.
https://alarm-redist.github.io/redist/
GNU General Public License v2.0
66 stars 23 forks source link

Helper or `filter` argument for submaps #156

Closed christopherkenny closed 1 year ago

christopherkenny commented 1 year ago

Let's make it a little easier to update populations without using attr<-() ...

CoryMcCartan commented 1 year ago

more detail?

christopherkenny commented 1 year ago

The idea here would be that if you have a redist_map and want to subset it, you could set something like .preserve_pop = TRUE within the filter if the population bounds should be preserved as-is. (Maybe it would be named .update_pop = TRUE by default if that better conveys what would happen.) Right now it tries to do it intelligently using the existing_col, but that isn't always the intended behavior.

CoryMcCartan commented 1 year ago

Addressed by fff0dc4a