SDV icon indicating copy to clipboard operation
SDV copied to clipboard

Add `random_state` arguments wherever relevant

Open csala opened this issue 4 years ago • 1 comments

There have been a few questions, such as #515 and #299 about how to set a random state for the models. External workarounds exist, like the one shared in those questions, but we may want to add random_state arguments that take care of doing this internally.

The arguments should be added in all the places where it makes sense doing so, such as the __init__, fit and sample methods of the models, or any other steps that involve any kind of randomness, such as the Constraints or the RDT transformers.

csala avatar Sep 09 '21 11:09 csala

Note that as of SDV 0.15.0, all single-table sampling functions now have a randomize_samples parameter to control the randomness.

Remaining places to control randomness are:

  • Multi-table sampling functions
  • Sequential model sampling functions
  • All fitting functions

npatki avatar Jul 07 '22 21:07 npatki