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

Tidy Constraint Checker -- Better Errors #96

Closed christopherkenny closed 2 years ago

christopherkenny commented 2 years ago

New constraints take a list input of the form:

constraints = list(vra = list(strength = ...))

The double list allows for flexibility on our end for extending options and cleaner input. However, if you forget the inner list, because we want it to work always, it doesn't error out or even warn.

christopherkenny commented 2 years ago

Should also point to errors when there are expectations about constraints that don't exist in SMC but do elsewhere. (#99)

CoryMcCartan commented 2 years ago

Fixed by our new constraints interface d3b3ce8