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

redist_shortbust no longer working in dev #114

Closed kuriwaki closed 2 years ago

kuriwaki commented 2 years ago

In the dev version I installed yesterday (2021-11-07), shortburst errors out. I can replicate the error message with the man page example below.

It works fine in the redist package dev version as of 251b9dbc7468cb7360700aecb7fd693a8a685408 (2021-11-03).

library(redist) # version 5949b (2021-11-07)
#> Warning: package 'redist' was built under R version 4.0.5
#> 
#> Attaching package: 'redist'
#> The following object is masked from 'package:stats':
#> 
#>     filter

data(iowa)

iowa_map = redist_map(iowa, existing_plan=cd_2010, pop_tol=0.01)
redist_shortburst(iowa_map, scorer_frac_kept(iowa_map), max_bursts=50)
#> MERGE-SPLIT SHORT BURSTS
#> Sampling up to 50 bursts of 10 iterations each.
#> Burst  Improve?  Score
#> Error in ms_plans(burst_size + 1L, adj, init, counties, pop, ndists, pop_bounds[2], : Expecting a single value: [extent=0].

Created on 2021-11-08 by the reprex package (v2.0.1)

Can include my sessionInfo if it's local depenendent. @CoryMcCartan @christopherkenny

CoryMcCartan commented 2 years ago

Thanks; this was caused by a change in verbosity reporting levels. Now fixed