`cluster_gen()` fails if both `cat_prop` and `rho` are set

[wleoncio]

MRE

cluster_gen(n = c(5, 2), n_X = 2, n_W = 2, cat_prop = list(c(0.5, 1), c(0.5, 1))) # OK
cluster_gen(n = c(5, 2), n_X = 2, n_W = 2, cat_prop = list(c(0.5, 1), c(0.5, 1)), rho = 0.5) # Fails 

Fixing this might be as simple as adding an else statement here, but the issue should be investigated further for side effects.

[wleoncio]

Hi @mollyolaf,

Please try the current development version of lsasim for a fix to this issue and let me know if the solution looks adequate:

remotes::install_github("tmatta/lsasim@develop")

As a minor improvement, version 9.1.5.9001 also improved the output of cluster_gen() so that something like "Total respondents: 10 + 10 + 10 + 10" is replaced with "10 * 4" (useful for large group numbers). :)

[wleoncio]

Hi @mollyolaf!

Happy New Year! I wonder if you had time to test the fixes to cluster_gen() on lsasim 2.1.5.9003. Let me know if we're clear to publish the update to CRAN or if there's something else to work on before that.

Best,
Waldir

[wleoncio]

Fixed on 83f54b0