More flexible `add_vary_across()`?
jpdunc23 opened this issue · 2 comments
jpdunc23 commented
Currently, the following is an error, because add_vary_across
was previously used for x
:
simChef/tests/testthat/test-experiment.R
Lines 785 to 787 in b3d71a2
Would it be better to append 1:3
to the list of variations on x
instead?
tiffanymtang commented
I would prefer to keep this as an error. It could get confusing to an outside reader to keep track of all the parameter values if they get added at various points in the code. Returning an error would hopefully help to encourage cleaner code.
jpdunc23 commented
Fair point! Thanks.