tmatta / lsasim

Simulate large scale assessment data
6 stars 5 forks source link

Consider changing handling of NULL parameters in questionnaire_gen #32

Closed wleoncio closed 3 years ago

wleoncio commented 3 years ago

cluster_gen and questionnaire_gen are two functions that contain lots of parameters which default to NULL. This is fine here and there, but too much of it can be a bit polluting. An alternative would be to not use default values and replacing is.null() calls with missing(). The problem with this approach is that it confuses the user about which arguments are mandatory and which aren't.

wleoncio commented 3 years ago

Leaving this as is, for now.